Page MenuHomePhabricator

[WarnMissedTransforms] Do not warn about already vectorized loops.
ClosedPublic

Authored by Meinersbur on Jan 31 2019, 1:39 PM.

Details

Summary

LoopVectorize adds llvm.loop.isvectorized, but leaves llvm.loop.vectorize.enable. Do not consider such a loop for user-forced vectorization since vectorization already happened by prioritizing llvm.loop.isvectorized except for TM_SuppressedByUser.

Fixes http://llvm.org/PR40546

Diff Detail

Repository
rL LLVM

Event Timeline

Meinersbur created this revision.Jan 31 2019, 1:39 PM
hfinkel accepted this revision.Feb 4 2019, 9:53 AM

LGTM.

This revision is now accepted and ready to land.Feb 4 2019, 9:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2019, 9:53 AM

LGTM.

And I think it would be good to get this into 8.0 to avoid spurious new user-visible warnings.

This revision was automatically updated to reflect the committed changes.