Check dangling constant users for Function/Global symbols, before iterating through the user list.

dlei6g · sys_zuul · commit 0176c6d44765 · 2020-04-07T16:29:55.000-07:00
Change-Id: I09d2d5c52c1ecb062aecf1f998b47693ddb61610
diff --git a/IGC/Compiler/CISACodeGen/EmitVISAPass.cpp b/IGC/Compiler/CISACodeGen/EmitVISAPass.cpp
@@ -9977,7 +9977,8 @@ void EmitPass::emitSymbolRelocation(Function& F)
             }
             else if (Constant* C = dyn_cast<Constant>(*it))
             {
-                return ValueUsedInFunction(C, currFunc);
+                if (C->isConstantUsed())
+                    return ValueUsedInFunction(C, currFunc);
             }
         }
         return false;

Original file line number	Diff line number	Diff line change
`@@ -9977,7 +9977,8 @@ void EmitPass::emitSymbolRelocation(Function& F)`
`9977`	`9977`	`}`
`9978`	`9978`	`else if (Constant* C = dyn_cast<Constant>(*it))`
`9979`	`9979`	`{`
`9980`		`- return ValueUsedInFunction(C, currFunc);`
	`9980`	`+ if (C->isConstantUsed())`
	`9981`	`+ return ValueUsedInFunction(C, currFunc);`
`9981`	`9982`	`}`
`9982`	`9983`	`}`
`9983`	`9984`	`return false;`