[Autobackout][FuncReg]Revert of change: 7d079ed

Add analysis of stateless memory load/store

Add the analysis to StatelessToStatefull pass to check if there is
non kernel argument memory load/store.

Change-Id: I0217a6a0d2e755cfe4b509664290ac67cc7974ab

Author: DianaChen

IGC/Compiler/CISACodeGen/helper.cpp

@@ -1378,9 +1378,9 @@ namespace IGC
             opcode == llvm_gradientYfine);
     }
 
-    bool IsStatelessMemLoadIntrinsic(llvm::GenISAIntrinsic::ID id)
+    bool IsStatelessMemLoadIntrinsic(const llvm::GenIntrinsicInst& inst)
     {
-        switch(id)
+        switch(inst.getIntrinsicID())
         {
         case GenISAIntrinsic::GenISA_simdBlockRead:
                 return true;
@@ -1390,9 +1390,9 @@ namespace IGC
         return false;
     }
 
-    bool IsStatelessMemStoreIntrinsic(llvm::GenISAIntrinsic::ID id)
+    bool IsStatelessMemStoreIntrinsic(const llvm::GenIntrinsicInst& inst)
     {
-        switch (id) {
+        switch (inst.getIntrinsicID()) {
         case GenISAIntrinsic::GenISA_simdBlockWrite:
             return true;
         default:
@@ -1401,7 +1401,7 @@ namespace IGC
         return false;
     }
 
-    bool IsStatelessMemAtomicIntrinsic(GenIntrinsicInst& inst, GenISAIntrinsic::ID id)
+    bool IsStatelessMemAtomicIntrinsic(const llvm::GenIntrinsicInst& inst)
     {
         // This includes:
         // GenISA_intatomicraw

IGC/Compiler/CISACodeGen/helper.h

@@ -175,9 +175,9 @@ namespace IGC
     bool IsSIMDBlockIntrinsic(llvm::Instruction* inst);
     bool isSubGroupIntrinsic(const llvm::Instruction* I);
 
-    bool IsStatelessMemLoadIntrinsic(llvm::GenISAIntrinsic::ID id);
-    bool IsStatelessMemStoreIntrinsic(llvm::GenISAIntrinsic::ID id);
-    bool IsStatelessMemAtomicIntrinsic(llvm::GenIntrinsicInst& inst, llvm::GenISAIntrinsic::ID id);
+    bool IsStatelessMemLoadIntrinsic(const llvm::GenIntrinsicInst& inst);
+    bool IsStatelessMemStoreIntrinsic(const llvm::GenIntrinsicInst& inst);
+    bool IsStatelessMemAtomicIntrinsic(const llvm::GenIntrinsicInst& inst);
 
     bool isURBWriteIntrinsic(const llvm::Instruction* inst);
 

IGC/Compiler/Optimizer/OpenCLPasses/StatelessToStatefull/StatelessToStatefull.cpp

@@ -604,22 +604,6 @@ void StatelessToStatefull::visitCallInst(CallInst& I)
                 m_changed = true;
             }
         }
-        // check if there's non-kernel-arg load/store
-        if (IsStatelessMemStoreIntrinsic(intrinID) ||
-            IsStatelessMemLoadIntrinsic(intrinID)  ||
-            IsStatelessMemAtomicIntrinsic(*Inst, intrinID)) {
-
-            Value* ptr = Inst->getOperand(0);
-            if (!pointerIsFromKernelArgument(*ptr)) {
-                CodeGenContext* ctx = getAnalysis<CodeGenContextWrapper>().getCodeGenContext();
-                if (IsStatelessMemStoreIntrinsic(intrinID))
-                    ctx->m_hasNonKernelArgStore = true;
-                else if (IsStatelessMemLoadIntrinsic(intrinID))
-                    ctx->m_hasNonKernelArgLoad = true;
-                else
-                    ctx->m_hasNonKernelArgAtomic = true;
-            }
-        }
     }
 }
 
@@ -668,12 +652,6 @@ void StatelessToStatefull::visitLoadInst(LoadInst& I)
 
         m_changed = true;
     }
-
-    // check if there's non-kernel-arg load/store
-    if (!pointerIsFromKernelArgument(*ptr)) {
-        CodeGenContext* ctx = getAnalysis<CodeGenContextWrapper>().getCodeGenContext();
-        ctx->m_hasNonKernelArgLoad = true;
-    }
 }
 
 void StatelessToStatefull::visitStoreInst(StoreInst& I)
@@ -715,11 +693,6 @@ void StatelessToStatefull::visitStoreInst(StoreInst& I)
             m_changed = true;
         }
     }
-
-    if (!pointerIsFromKernelArgument(*ptr)) {
-        CodeGenContext* ctx = getAnalysis<CodeGenContextWrapper>().getCodeGenContext();
-        ctx->m_hasNonKernelArgStore = true;
-    }
 }
 
 CallInst* StatelessToStatefull::createBufferPtr(unsigned addrSpace, Constant* argNumber, Instruction* InsertBefore)