This is an archive of the discontinued LLVM Phabricator instance.

[NewPM] Consistently use 'simplifycfg' rather than 'simplify-cfg'
ClosedPublic

Authored by bjope on Jul 8 2021, 6:23 AM.

Details

Summary

There was an alias between 'simplifycfg' and 'simplify-cfg' in the
PassRegistry. That was the original reason for this patch, which
effectively removes the alias.

This patch also replaces all occurrances of 'simplify-cfg'
by 'simplifycfg'. Reason for choosing that form for the name is
that it matches the DEBUG_TYPE for the pass, and the legacy PM name
and also how it is spelled out in other passes such as
'loop-simplifycfg', and in other options such as
'simplifycfg-merge-cond-stores'.

I for some reason the name should be changed to 'simplify-cfg' in
the future, then I think such a renaming should be more widely done
and not only impacting the PassRegistry.

Diff Detail

Event Timeline

bjope created this revision.Jul 8 2021, 6:23 AM
bjope requested review of this revision.Jul 8 2021, 6:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 8 2021, 6:23 AM
aeubanks accepted this revision.Jul 8 2021, 8:48 AM
This revision is now accepted and ready to land.Jul 8 2021, 8:48 AM
This revision was landed with ongoing or failed builds.Jul 9 2021, 12:59 AM
This revision was automatically updated to reflect the committed changes.