DebugInfo: return empty variable location in case llvm.dbg.declare contains invalid address.

PawelJurek · sys_zuul · commit 9149da480258 · 2019-11-07T05:55:46.000-08:00
Change-Id: I73c2d068bcfda90894b0851c4f58887c2e467e84
diff --git a/IGC/Compiler/DebugInfo/VISAModule.cpp b/IGC/Compiler/DebugInfo/VISAModule.cpp
@@ -518,9 +518,11 @@ VISAVariableLocation VISAModule::GetVariableLocation(const llvm::Instruction* pI
     Type* pType = pValue->getType();
     if (isDbgDclInst)
     {
-        assert(pType->isPointerTy() && "DBG declare intrinsic must point to an address");
+        if (!pType->isPointerTy()) {
+            assert(0 && "DBG declare intrinsic must point to an address");
+            return VISAVariableLocation();
+        }
         pType = pType->getPointerElementType();
-
     }
 
     bool isInSurface = false;

Original file line number	Diff line number	Diff line change
`@@ -518,9 +518,11 @@ VISAVariableLocation VISAModule::GetVariableLocation(const llvm::Instruction* pI`
`518`	`518`	`Type* pType = pValue->getType();`
`519`	`519`	`if (isDbgDclInst)`
`520`	`520`	`{`
`521`		`- assert(pType->isPointerTy() && "DBG declare intrinsic must point to an address");`
	`521`	`+ if (!pType->isPointerTy()) {`
	`522`	`+ assert(0 && "DBG declare intrinsic must point to an address");`
	`523`	`+ return VISAVariableLocation();`
	`524`	`+ }`
`522`	`525`	`pType = pType->getPointerElementType();`
`523`		`-`
`524`	`526`	`}`
`525`	`527`
`526`	`528`	`bool isInSurface = false;`