[PATCH] D55641: [CallSite removal] Migrate all Alias Analysis APIs to use the newly minted `CallBase` class instead of the `CallSite` wrapper.

George Burgess IV via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 15 23:42:31 PST 2018


george.burgess.iv added a comment.

MemorySSA changes LGTM, modulo one comment. Thanks!



================
Comment at: llvm/lib/Analysis/MemorySSA.cpp:133
   MemoryLocOrCall(Instruction *Inst) {
-    if (ImmutableCallSite(Inst)) {
+    if (auto *C = dyn_cast<CallInst>(Inst)) {
       IsCall = true;
----------------
s/CallInst/CallBase/?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55641





More information about the llvm-commits mailing list