Page MenuHomePhabricator

[NewPM] Add Option handling for SimplifyCFG
ClosedPublic

Authored by skatkov on Apr 14 2019, 10:47 PM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

skatkov created this revision.Apr 14 2019, 10:47 PM
fedor.sergeev added inline comments.Apr 14 2019, 11:02 PM
lib/Passes/PassBuilder.cpp
1441 ↗(On Diff #195095)

I believe you do not want to allow "no-" prefix for this option.
Add Enable to condition?

skatkov marked an inline comment as done.Apr 14 2019, 11:05 PM
skatkov added inline comments.
lib/Passes/PassBuilder.cpp
1441 ↗(On Diff #195095)

Make sense. Will fix that before landing if there is no other objections.

This revision is now accepted and ready to land.Apr 14 2019, 11:26 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 15 2019, 1:59 AM