[PATCH] D78470: [CallSite removal][SCCP] Use CallBase instead of CallSite. NFC

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 19 23:25:38 PDT 2020


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


================
Comment at: llvm/lib/Transforms/Scalar/SCCP.cpp:1134
 
-void SCCPSolver::visitCallSite(CallSite CS) {
-  handleCallResult(CS);
-  handleCallArguments(CS);
+void SCCPSolver::visitCallBase(CallBase &I) {
+  handleCallResult(I);
----------------
mtrofin wrote:
> Nit: here and below, CallBase &CB?
I was just trying to be consistent with the other visitor functions in the file. But I can change it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78470





More information about the llvm-commits mailing list