[PATCH] D146599: [InstSimplify] Actually use NewOps for calls in simplifyInstructionWithOperands
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 22 01:11:40 PDT 2023
nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:6619-6621
+ SmallVector<Value *, 4> Args;
+ for (Value *O : Call->args())
+ Args.push_back(O);
----------------
Should work nowadays, I think.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146599/new/
https://reviews.llvm.org/D146599
More information about the llvm-commits
mailing list