[Autobackout][FuncReg]Revert of change: d65d5cb

fangliu2020 · fangliu2020 · commit b25b5ebe8b4d · 2020-02-21T21:52:18.000-08:00
Fix the bug of no stateless-to-stateful converting if no GEP existed.

Change-Id: Ifa1eb9463f922ffee0794a867d4c03bd72ebdaf3
diff --git a/IGC/Compiler/Optimizer/OpenCLPasses/StatelessToStatefull/StatelessToStatefull.cpp b/IGC/Compiler/Optimizer/OpenCLPasses/StatelessToStatefull/StatelessToStatefull.cpp
@@ -362,7 +362,7 @@ bool StatelessToStatefull::pointerIsPositiveOffsetFromKernelArgument(
 
     // stripPointerCasts might skip addrSpaceCast, thus check if AS is still
     // the original one. Also, if base is still instruction, skip.
-    if (cast<PointerType>(base->getType())->getAddressSpace() == ptrAS && !isa<Instruction>(base))
+    if (gep && cast<PointerType>(base->getType())->getAddressSpace() == ptrAS && !isa<Instruction>(base))
     {
         if (const KernelArg * arg = getKernelArg(base))
         {

Original file line number	Diff line number	Diff line change
`@@ -362,7 +362,7 @@ bool StatelessToStatefull::pointerIsPositiveOffsetFromKernelArgument(`
`362`	`362`
`363`	`363`	`// stripPointerCasts might skip addrSpaceCast, thus check if AS is still`
`364`	`364`	`// the original one. Also, if base is still instruction, skip.`
`365`		`- if (cast<PointerType>(base->getType())->getAddressSpace() == ptrAS && !isa<Instruction>(base))`
	`365`	`+ if (gep && cast<PointerType>(base->getType())->getAddressSpace() == ptrAS && !isa<Instruction>(base))`
`366`	`366`	`{`
`367`	`367`	`if (const KernelArg * arg = getKernelArg(base))`
`368`	`368`	`{`