This is an archive of the discontinued LLVM Phabricator instance.

[SLPVectorizer] Set flag to previous default.
ClosedPublic

Authored by asbirlea on May 15 2019, 10:58 AM.

Details

Summary

The refactoring in r360276 moved the RunSLPVectorization flag and added the default explicitly. The default should have been false, as before.

The new pass manager used to have SLPVectorization on by default, now it's off in opt, and needs D61617 checked in to enable it in clang.

Diff Detail

Repository
rL LLVM

Event Timeline

asbirlea created this revision.May 15 2019, 10:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2019, 10:58 AM
chandlerc accepted this revision.May 17 2019, 2:52 PM

We should get folks to figure out why this regresses them, but making the two PMs behave the same seems fine for now.

This revision is now accepted and ready to land.May 17 2019, 2:52 PM
This revision was automatically updated to reflect the committed changes.