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

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 15 23:56:30 PST 2018


chandlerc marked 2 inline comments as done.
chandlerc added inline comments.


================
Comment at: llvm/lib/Analysis/MemorySSA.cpp:133
   MemoryLocOrCall(Instruction *Inst) {
-    if (ImmutableCallSite(Inst)) {
+    if (auto *C = dyn_cast<CallInst>(Inst)) {
       IsCall = true;
----------------
george.burgess.iv wrote:
> s/CallInst/CallBase/?
Good catch, fixed!


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