[PATCH] D60636: [CallSite removal] move InlineCost to CallBase usage

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 22 09:26:05 PDT 2019


reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Analysis/InlineCost.cpp:2095
+      if (!ReturnsTwice && isa<CallInst>(Call) &&
+          cast<CallInst>(Call)->canReturnTwice())
         return "exposes returns-twice attribute";
----------------
Doesn't look like you need the cast to CallInst here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60636





More information about the llvm-commits mailing list