[PATCH] D78219: [CallSite removal][SampleProfile] Use CallBase instead of CallSite. NFC

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 12:03:36 PDT 2020


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

Looks good - if you like, as a follow-up, it'd be great to change some of these function/data structures to use CallBase* rather than Instruction* now that that's an option (the functions that unconditionally cast<CallBase> their parameters, or for loops that unconditionally cast<CallBase> the elements, etc - perhaps could use CallBase parameter/element types to begin with)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78219





More information about the llvm-commits mailing list