Don't compare pointer types on opaque pointers in

karolzwolak · igcbot · commit a6a31604156a · 2025-10-08T09:53:13.000+02:00
`SOALayoutChecker::visitBitCastInst()`

`SOALayoutChecker::visitBitCastInst()` assumed we're on typed pointers
and tried to get pointer types, triggering LLVM assert on opaque
pointers.
diff --git a/IGC/Compiler/CISACodeGen/LowerGEPForPrivMem.cpp b/IGC/Compiler/CISACodeGen/LowerGEPForPrivMem.cpp
@@ -517,10 +517,11 @@ bool SOALayoutChecker::checkUsers(Instruction &I) {
 }
 
 bool SOALayoutChecker::visitBitCastInst(BitCastInst &BI) {
-  if (BI.use_empty() || IsBitCastForLifetimeMark(&BI)) {
+  // no sense in comparing pointer base types on opaque pointers
+  if (BI.use_empty() || IsBitCastForLifetimeMark(&BI) || AreOpaquePointersEnabled()) {
     return true;
   }
-
+  // FIXME: remove this once we only support opaque pointers
   Type *baseT = GetBaseType(IGCLLVM::getNonOpaquePtrEltTy(BI.getType()), true);
   Type *sourceType = GetBaseType(IGCLLVM::getNonOpaquePtrEltTy(BI.getOperand(0)->getType()), true);
   if (baseT->isStructTy() || sourceType->isStructTy()) {

Original file line number	Diff line number	Diff line change
`@@ -517,10 +517,11 @@ bool SOALayoutChecker::checkUsers(Instruction &I) {`
`517`	`517`	`}`
`518`	`518`
`519`	`519`	`bool SOALayoutChecker::visitBitCastInst(BitCastInst &BI) {`
`520`		`- if (BI.use_empty() \|\| IsBitCastForLifetimeMark(&BI)) {`
	`520`	`+ // no sense in comparing pointer base types on opaque pointers`
	`521`	`+ if (BI.use_empty() \|\| IsBitCastForLifetimeMark(&BI) \|\| AreOpaquePointersEnabled()) {`
`521`	`522`	`return true;`
`522`	`523`	`}`
`523`		`-`
	`524`	`+ // FIXME: remove this once we only support opaque pointers`
`524`	`525`	`Type *baseT = GetBaseType(IGCLLVM::getNonOpaquePtrEltTy(BI.getType()), true);`
`525`	`526`	`Type *sourceType = GetBaseType(IGCLLVM::getNonOpaquePtrEltTy(BI.getOperand(0)->getType()), true);`
`526`	`527`	`if (baseT->isStructTy() \|\| sourceType->isStructTy()) {`