morehouse added a comment.

At this point I am not convinced this patch will provide benefit for the 
default use case when `-entropic=1`.  I am hesitant to add complexity to the 
code for unsure benefit.

If you request a FuzzBench experiment 
<https://google.github.io/fuzzbench/getting-started/adding-a-new-fuzzer/#requesting-an-experiment>
 to get some data on this, and the results look good, then I'll be willing to 
invest more time into reviewing this patch.

Please CC me on the FuzzBench pull request, so I can make sure we are 
evaluating this properly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100161

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to