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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri marked an inline comment as done.
lebedev.ri 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");
----------------
mtrofin wrote:
> 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.
(It might be a really good idea to do all local development with `LLVM_ENABLE_ASSERTIONS` enabled..)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78538