[llvm] InferAddressSpaces: Make getPredicatedAddrSpace less confusing (PR #104052)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 14 10:48:26 PDT 2024


================
@@ -989,7 +986,8 @@ bool InferAddressSpacesImpl::updateAddressSpace(
           OperandAS = PtrOperand->getType()->getPointerAddressSpace();
           if (OperandAS == FlatAddrSpace) {
             // Check AC for assumption dominating V.
-            unsigned AS = getPredicatedAddrSpace(V, PtrOperand);
+            unsigned AS =
+                getPredicatedAddrSpace(*PtrOperand, &cast<Instruction>(V));
----------------
arsenm wrote:

This is currently only used from the result of getPointerOperands, so it must at least be Operator. It could maybe be  a constantexpr, but I'm not sure it's possible 

https://github.com/llvm/llvm-project/pull/104052


More information about the llvm-commits mailing list