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

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 19 22:57:08 PDT 2020


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

lgtm, some naming nits



================
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);
----------------
Nit: here and below, CallBase &CB?


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