Page MenuHomePhabricator

Deprecation flag group + use for BB vectorizer options
ClosedPublic

Authored by joerg on Jul 1 2017, 9:48 AM.

Details

Summary

As discussed on IRC when D34846 came up, removing flags is problematic from a UX experience.

This change introduces a new option group similar to the group GCC optimizer flags like -fexpensive-optimizations. This gives a warning by default, but can be silenced appropriately. Move the two BB vectorizer flags into this option group, test them and add a release note entry.

Diff Detail

Repository
rL LLVM

Event Timeline

joerg created this revision.Jul 1 2017, 9:48 AM
hfinkel accepted this revision.Jul 1 2017, 1:28 PM
hfinkel added a subscriber: hfinkel.

LGTM

docs/ReleaseNotes.rst
69 ↗(On Diff #104998)

There was only one pass, so maybe say, "... to enable the basic-block-vectorization pass."

This revision is now accepted and ready to land.Jul 1 2017, 1:28 PM
This revision was automatically updated to reflect the committed changes.