This is an archive of the discontinued LLVM Phabricator instance.

[llvm-opt-fuzzer] Add couple of popular passes
ClosedPublic

Authored by igor-laevsky on Jan 23 2018, 3:10 AM.

Details

Summary

I think it's time to extend passes covered by the llvm-opt-fuzzer. As a first batch I have chosen couple of the most popular non-loop passes.

Diff Detail

Repository
rL LLVM

Event Timeline

igor-laevsky created this revision.Jan 23 2018, 3:10 AM
bogner accepted this revision.Jan 23 2018, 11:05 AM

No complaints here. One minor question about the implementation.

lib/FuzzMutate/FuzzerCLI.cpp
87–88 ↗(On Diff #131022)

Maybe we've already gone over this, but why is this startswith() rather than equals()?

This revision is now accepted and ready to land.Jan 23 2018, 11:05 AM
This revision was automatically updated to reflect the committed changes.
igor-laevsky added inline comments.Jan 24 2018, 2:00 AM
lib/FuzzMutate/FuzzerCLI.cpp
87–88 ↗(On Diff #131022)

Actually, no good reason, need to fix that :) Thanks!