Page MenuHomePhabricator

[ARM] Fix selection of VLDR.16 instruction with imm offset
ClosedPublic

Authored by olista01 on Mar 1 2019, 5:54 AM.

Details

Summary

The isScaledConstantInRange function takes upper and lower bounds which are checked after dividing by the scale, so the bounds checks for half, single and double precision should all be the same. Previously, we had wrong bounds checks for half precision, so selected an immediate the instructions can't actually represent.

Diff Detail

Repository
rL LLVM

Event Timeline

olista01 created this revision.Mar 1 2019, 5:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 1 2019, 5:54 AM
This revision is now accepted and ready to land.Mar 1 2019, 12:17 PM
This revision was automatically updated to reflect the committed changes.