[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