[PATCH] D94813: [MergeICmps] Don't merge icmps derived from pointers with addressspaces

[Alexander Richardson via Phabricator via llvm-commits]

arichardson added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/MergeICmps.cpp:148
   Value *const Addr = LoadI->getOperand(0);
+  if (Addr->getType()->getPointerAddressSpace() != 0) {
+    LLVM_DEBUG(dbgs() << "from non-zero AddressSpace\n");
----------------
In the CHERI-enabled targets (for Aarch64,RISC-V and MIPS) memcmp takes `addrspace(200)` arguments. Would it be possible to use the memcmp signature instead of a hardcoded AS0 check?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94813/new/

https://reviews.llvm.org/D94813