[PATCH] D57494: [CallSite removal] Remove CallSite uses from InstCombine.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 30 19:52:01 PST 2019


craig.topper marked an inline comment as done.
craig.topper added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:4263
   // TODO: Do the transform if it only requires adding pointer casts.
-  if (CS.isMustTailCall())
+  if (isa<CallInst>(CB) && cast<CallInst>(CB).isMustTailCall())
     return false;
----------------
Wasn't sure what to do here. isMustTailCall is inlined in CallSite, but it can't be inlined in CallBase.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57494





More information about the llvm-commits mailing list