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.
Paths
| Differential D66222
[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 Event TimelineThis revision is now accepted and ready to land.Aug 15 2019, 2:37 AM Closed by commit rL368996: [ARM] Fix alignment checks for BE VLDRH (authored by dmgreen). · Explain WhyAug 15 2019, 5:55 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 215130 llvm/lib/Target/ARM/ARMInstrMVE.td
llvm/test/CodeGen/Thumb2/mve-ldst-offset.ll
llvm/test/CodeGen/Thumb2/mve-ldst-postinc.ll
llvm/test/CodeGen/Thumb2/mve-ldst-preinc.ll
|