[llvm] [llvm][SelectionDAG] Relax llvm.ptrmask's size check on arm64_32 (PR #94125)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 2 23:20:13 PDT 2024


================
@@ -7839,12 +7839,26 @@ void SelectionDAGBuilder::visitIntrinsicCall(const CallInst &I,
     return;
   }
   case Intrinsic::ptrmask: {
+    unsigned PtrBits =
+        DAG.getDataLayout().getIndexTypeSizeInBits(I.getOperand(0)->getType());
----------------
nikic wrote:

This should be checking the pointer size, not the index size. The implementation below is only correct as long as these are the same. (The index size and the mask size are known to be the same by verifier enforcement.)

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


More information about the llvm-commits mailing list