This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Fix alignment checks for BE VLDRH
ClosedPublic

Authored by dmgreen on Aug 14 2019, 8:33 AM.

Details

Summary

We need to allow any alignment at least 2, not just exactly 2, so that the big endian loads and stores can be selected successfully. I've also added extra BE testing for the load and store tests.

Thanks to Oliver for the report.

Diff Detail

Repository
rL LLVM

Event Timeline

dmgreen created this revision.Aug 14 2019, 8:33 AM
This revision is now accepted and ready to land.Aug 15 2019, 2:37 AM
This revision was automatically updated to reflect the committed changes.