[PATCH] D139067: [FuzzMutate] New InsertCFGStrategy

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/FuzzMutate/IRMutator.cpp:341
+    auto RS =
+        makeSampler(IB.Rand, make_filter_range(IB.KnownTypes, isIntegerType));
+    assert(RS && "I can't find an integer type in all allowed types, is the "
----------------
Don't name the lambda?


================
Comment at: llvm/lib/FuzzMutate/IRMutator.cpp:344
+                 "setting correct?");
+    IntegerType *Ty = static_cast<IntegerType *>(RS.getSelection());
+    uint64_t BitSize = Ty->getBitWidth();
----------------
cast<>? I don't see  what RS really is


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139067