[PATCH] D100161: Redistribute energy for Corpus

2021-07-02 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. I redesigned the algorithm and did a complete long-term evaluation by myself, and got very good results. Whether it is -entropic=0 or 1, it performs very well, and -fork mode is now better than paralllel fuzzing mode Better performance, please move to D105084. Th

[PATCH] D100161: Redistribute energy for Corpus

2021-04-14 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2689156 , @morehouse wrote: > 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 y

[PATCH] D100161: Redistribute energy for Corpus

2021-04-14 Thread Matt Morehouse via Phabricator via cfe-commits
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

[PATCH] D100161: Redistribute energy for Corpus

2021-04-14 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. Hello. Due to the time zone difference, I think our communication is a bit inefficient. Can we arrange a convenient time for you to focus on the discussion? We use CST. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/

[PATCH] D100161: Redistribute energy for Corpus

2021-04-13 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2686130 , @morehouse wrote: > If the effect is similar to entropic, why do we need this patch as well? They just have some similarities, they will be better after patching. Repository: rG LLVM Github Monorepo CHAN

[PATCH] D100161: Redistribute energy for Corpus

2021-04-13 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse added a comment. If the effect is similar to entropic, why do we need this patch as well? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100161/new/ https://reviews.llvm.org/D100161 ___ cfe-comm

[PATCH] D100161: Redistribute energy for Corpus

2021-04-13 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. Maybe uniform-random approach change efficacy! 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

[PATCH] D100161: Redistribute energy for Corpus

2021-04-13 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. If not sorted by size , Just a simple grouping of corpus, the effect is similar to entropic. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100161/new/ https://reviews.llvm.org/D100161

[PATCH] D100161: Redistribute energy for Corpus

2021-04-12 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2683632 , @morehouse wrote: > Also, the descriptions states: > >> Divide the corpus into n parts according to size. > > Is it really according to size? IIUC when there are multiple worker > processes, any new coverage

[PATCH] D100161: Redistribute energy for Corpus

2021-04-12 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2683622 , @morehouse wrote: > Thanks for sharing your data. Took a quick look and seems promising. > > I would like to try this on FuzzBench before accepting the patch though. > FuzzBench has a very nice experimental

[PATCH] D100161: Redistribute energy for Corpus

2021-04-12 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2683632 , @morehouse wrote: > Also, the descriptions states: > >> Divide the corpus into n parts according to size. > > Is it really according to size? IIUC when there are multiple worker > processes, any new coverage

[PATCH] D100161: Redistribute energy for Corpus

2021-04-12 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse added a comment. Also, the descriptions states: > Divide the corpus into n parts according to size. Is it really according to size? IIUC when there are multiple worker processes, any new coverage they have simply gets appended to `Files`. So `Files` is not necessarily sorted by siz

[PATCH] D100161: Redistribute energy for Corpus

2021-04-12 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse added a subscriber: metzman. morehouse added a comment. Thanks for sharing your data. Took a quick look and seems promising. I would like to try this on FuzzBench before accepting the patch though. FuzzBench has a very nice experimental framework for evaluating changes like this. >

[PATCH] D100161: Redistribute energy for Corpus

2021-04-10 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2681045 , @gtt1995 wrote: > F16230733: experiment.tar > > This is part of raw data , the object from oss-fuzz project. The patched version data in average dir, data of libfuzzer di

[PATCH] D100161: Redistribute energy for Corpus

2021-04-10 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. F16230733: experiment.tar This is part of raw data , the object from oss-fuzz project. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100161/new/ https://reviews.llvm.org/D100161 __

[PATCH] D100161: Redistribute energy for Corpus

2021-04-10 Thread taotao gu via Phabricator via cfe-commits
gtt1995 added a comment. In D100161#2679922 , @morehouse wrote: > Thanks for the patch! Would you mind sharing the experimental data/results > you obtained for this patch? > > Additionally, could you submit this patch to FuzzBench >

[PATCH] D100161: Redistribute energy for Corpus

2021-04-09 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse added a comment. Thanks for the patch! Would you mind sharing the experimental data/results you obtained for this patch? Additionally, could you submit this patch to FuzzBench for an independent evaluation? Thanks, Matt Repository: rG LLVM G

[PATCH] D100161: Redistribute energy for Corpus

2021-04-08 Thread taotao gu via Phabricator via cfe-commits
gtt1995 created this revision. gtt1995 added reviewers: 01alchemist, 0b01. gtt1995 requested review of this revision. Herald added projects: clang, Sanitizers. Herald added subscribers: Sanitizers, cfe-commits. Divide the corpus into n parts according to size. Each job executes each corpus in tur