In StatelessToStatefull check for address-space. unity between

pszymich · gfxbot · commit 68c609e1e103 · 2019-01-14T17:59:25.000-08:00
original instruction and last. GetElementPointer while also
respecting simple pointers

Change-Id: I6308db3eeb67ac3be7224f12fc85a198c5078e3b
diff --git a/IGC/Compiler/Optimizer/OpenCLPasses/StatelessToStatefull/StatelessToStatefull.cpp b/IGC/Compiler/Optimizer/OpenCLPasses/StatelessToStatefull/StatelessToStatefull.cpp
@@ -353,7 +353,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 ((gep && gep->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
`@@ -353,7 +353,7 @@ bool StatelessToStatefull::pointerIsPositiveOffsetFromKernelArgument(`
`353`	`353`
`354`	`354`	`// stripPointerCasts might skip addrSpaceCast, thus check if AS is still`
`355`	`355`	`// the original one. Also, if base is still instruction, skip.`
`356`		`- if ((gep && gep->getAddressSpace() == ptrAS) && !isa<Instruction>(base))`
	`356`	`+ if (gep && cast<PointerType>(base->getType())->getAddressSpace() == ptrAS && !isa<Instruction>(base))`
`357`	`357`	`{`
`358`	`358`	`if (const KernelArg* arg = getKernelArg(base))`
`359`	`359`	`{`