emmettneyman updated this revision to Diff 160740.
emmettneyman added a comment.
Changed the multiloop protos to nested loop protos. All the protos have an
inner loop and an outer loop.
Repository:
rC Clang
https://reviews.llvm.org/D50670
Files:
clang/tools/clang-fuzzer/cxx_loop_proto.pro
morehouse added a comment.
Another option would be to allow simple control flow within the loop itself.
Repository:
rC Clang
https://reviews.llvm.org/D50670
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
morehouse added a comment.
In https://reviews.llvm.org/D50670#1199556, @emmettneyman wrote:
> Should I switch my focus to nested loops instead? I think nested loops will
> increase coverage.
Yes, I'd recommend doing that.
Repository:
rC Clang
https://reviews.llvm.org/D50670
___
emmettneyman added a comment.
No, it doesn't actually. I thought it would try to combine the separate loops
into one block of vectorized instructions but after looking at the coverage of
multiple loops vs single loop, they cover exactly the same parts of the Loop
Vectorizer. Should I switch my
morehouse added a comment.
Does having multiple loops one after another change any coverage in the
vectorizer?
Repository:
rC Clang
https://reviews.llvm.org/D50670
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
emmettneyman created this revision.
emmettneyman added reviewers: morehouse, kcc.
Herald added a subscriber: cfe-commits.
Extended `cxx_loop_proto` to have multiple for loops. Modified
`loop_proto_to_llvm` and `loop_proto_to_cxx` to handle the new protos. In
`loop_proto_to_llvm`, I only translat
