This is an archive of the discontinued LLVM Phabricator instance.

Fix LoopLoadElimination to keep original alignment on the inital hoisted store
ClosedPublic

Authored by mehdi_amini on Jan 6 2017, 12:38 PM.

Details

Summary

This is fixing a bug where Loop Vectorization is widening a load but
with a lower alignment. Hoisting the load without propagating the alignment
will allow inst-combine to later deduce a higher alignment that what the pointer
actually is.

Event Timeline

mehdi_amini retitled this revision from to Fix LoopLoadElimination to keep original alignment on the inital hoisted store.
mehdi_amini updated this object.
mehdi_amini added a reviewer: anemet.
mehdi_amini added a subscriber: llvm-commits.
anemet accepted this revision.Jan 6 2017, 1:13 PM
anemet edited edge metadata.

LGTM. Mehdi, thanks very much for fixing this.

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