[PATCH] D109134: [MemorySSA] Support invariant.group metadata

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Analysis/MemorySSA.cpp:2495
+  // current function.
+  if (isa<GlobalValue>(PointerOperand))
+    return nullptr;
----------------
Shouldn't this skip `isa<Constant>` in general? I think you could reach here with a constant expression (say an inttoptr expression).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109134