[PATCH] D139067: [FuzzMutate] New InsertCFGStrategy

Peter Rong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 15:17:11 PST 2022


Peter marked 6 inline comments as done.
Peter added inline comments.


================
Comment at: llvm/lib/FuzzMutate/IRMutator.cpp:317
+  SmallVector<Instruction *, 32> Insts;
+  for (auto I = BB.getFirstInsertionPt(), E = BB.end(); I != E; ++I)
+    Insts.push_back(&*I);
----------------
arsenm wrote:
> Can do this in the constructor Insts(BB.getFirstInsertionPt(), BB.end())
Unfortunately no, BB.getFirstInsertionPt returns an iterator, which can't be implicitly converted to `Instruction*`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139067



More information about the llvm-commits mailing list