[PATCH] D78322: [CallSite removal][IPO] Move AbstractCallSite to its own header and change it to use CallBase instead of CallSite. NFCI

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 16 13:23:36 PDT 2020


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

Looks good - I'd probably have done it differently (waited until all the CallSite uses were eliminated, deleted that code from CallSite.h, then renamed CallSite.h to AbstractCallSite.h) because I /think/ it'd present easier/better revision history, but that's only based on /really/ vague guesses about how smart/not-smart the change tracking stuff is - in any case, dealer's choice :)


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

https://reviews.llvm.org/D78322





More information about the llvm-commits mailing list