[PATCH] D78538: [llvm][NFC][CallSite] Remove CallSite from DeadArgumentElimination

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 21 11:52:37 PDT 2020


mtrofin marked 2 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/DeadArgumentElimination.cpp:444
 
-        assert(CS.getArgument(ArgNo)
-               == CS->getOperand(U->getOperandNo())
-               && "Argument is not where we expected it");
+        assert(CB->getArgument(ArgNo) == CB->getOperand(U->getOperandNo()) &&
+               "Argument is not where we expected it");
----------------
foad wrote:
> Needs to be `CB->getArgOperand`. This is making my DEBUG build fail.
Noted, doing a rebuild locally to validate there's no other failures and sending a fix asap. Sorry about this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78538





More information about the llvm-commits mailing list