Page MenuHomePhabricator

Allow llc to run passes under the new pass manager one at a time.
Needs ReviewPublic

Authored by czhang on Jul 3 2019, 6:39 PM.

Event Timeline

czhang created this revision.Jul 3 2019, 6:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 3 2019, 6:39 PM
arsenm added a subscriber: arsenm.Jul 3 2019, 7:22 PM
arsenm added inline comments.
llvm/tools/llc/llc.cpp
60

This header should be renamed to match the new pass name

209

-run-pass-new would be a slightly less confusing name

czhang updated this revision to Diff 208562.Jul 8 2019, 5:52 PM

rebase & run-new-pass > run-pass-new & clang-format

czhang marked 2 inline comments as done.Jul 8 2019, 5:56 PM
probinson added inline comments.
llvm/tools/llc/llc.cpp
62

Is there some reason these includes aren't in order with the others above?

czhang marked an inline comment as done.Jul 9 2019, 10:10 AM
czhang added inline comments.
llvm/tools/llc/llc.cpp
62

Yes, these are only needed for the included new pass manager registry. This diff is only meant to hack up llc to actually use the new pass manager, so I thought it would be better to keep these includes visually separate for now.

lenary added a subscriber: lenary.Jul 12 2019, 1:29 AM