[PATCH] D30684: [libFuzzer] Don't use same seed for all parallel jobs
Kostya Serebryany via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 7 11:33:46 PST 2017
kcc added a comment.
In https://reviews.llvm.org/D30684#694651, @vitalybuka wrote:
> In https://reviews.llvm.org/D30684#694594, @kcc wrote:
>
> > it's not easy to emulate -jobs in a script, because -jobs=N works together with -workers=M and if N > M new jos start after some old die.
>
>
> And user has to go this inconvenient way if jobs and seed both are desired.
You are the first user to request this :)
And you are also not really a user, but a developer :)
> I agree that
> convenience(jobs+seed) < convenience(jobs)
> complexity(jobs) < convenience(jobs)
> but also
> complexity(jobs+seed) < complexity(jobs)
>
> It's still not clear to my why complexity(jobs+seed) > convenience(jobs+seed)
With you change, you will also need to document the non-intuitive behavior.
In all such cases it's easier to not have non-intuitive behavior
https://reviews.llvm.org/D30684
More information about the llvm-commits
mailing list