Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Kostya Serebryany via cfe-commits
+klimek@ who contributed the first version of clang-fuzzer. We now have clang-proto-fuzzer (see tools/clang/tools/clang-fuzzer/README.txt) which shares code with clang-fuzzer, but which also depends on various packages (protobufs, in particular). clang-proto-fuzzer is off by default (requires a

Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Nico Weber via cfe-commits
On Thu, Aug 10, 2017 at 3:13 PM, Kostya Serebryany wrote: > > > On Thu, Aug 10, 2017 at 12:01 PM, Nico Weber wrote: > >> On Thu, Aug 10, 2017 at 2:04 PM, Kostya Serebryany >> wrote: >> >>> >>> >>> On Thu, Aug 10, 2017 at 10:56 AM, Nico

Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Kostya Serebryany via cfe-commits
On Thu, Aug 10, 2017 at 12:01 PM, Nico Weber wrote: > On Thu, Aug 10, 2017 at 2:04 PM, Kostya Serebryany wrote: > >> >> >> On Thu, Aug 10, 2017 at 10:56 AM, Nico Weber via cfe-commits < >> cfe-commits@lists.llvm.org> wrote: >> >>> I really believe this has

Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Nico Weber via cfe-commits
On Thu, Aug 10, 2017 at 2:04 PM, Kostya Serebryany wrote: > > > On Thu, Aug 10, 2017 at 10:56 AM, Nico Weber via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> I really believe this has way too many deps to live in the clang repro, >> as said on the review already. >> >

Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Kostya Serebryany via cfe-commits
On Thu, Aug 10, 2017 at 10:56 AM, Nico Weber via cfe-commits < cfe-commits@lists.llvm.org> wrote: > I really believe this has way too many deps to live in the clang repro, as > said on the review already. > I don't have a very strong opinion here and would be happy to move if I see more support

Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Matt Morehouse via cfe-commits
+ Kostya The project isn't built by default, so the dependencies aren't needed unless you build the fuzzer specifically. On Thu, Aug 10, 2017 at 10:56 AM, Nico Weber wrote: > I really believe this has way too many deps to live in the clang repro, as > said on the review

Re: r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-10 Thread Nico Weber via cfe-commits
I really believe this has way too many deps to live in the clang repro, as said on the review already. Maybe this could live in clang-extra instead? On Aug 8, 2017 4:15 PM, "Matt Morehouse via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Author: morehouse > Date: Tue Aug 8 13:15:04 2017

r310408 - Integrate Kostya's clang-proto-fuzzer with LLVM.

2017-08-08 Thread Matt Morehouse via cfe-commits
Author: morehouse Date: Tue Aug 8 13:15:04 2017 New Revision: 310408 URL: http://llvm.org/viewvc/llvm-project?rev=310408=rev Log: Integrate Kostya's clang-proto-fuzzer with LLVM. Summary: The clang-proto-fuzzer models a subset of C++ as a protobuf and uses libprotobuf-mutator to generate