This is an archive of the discontinued LLVM Phabricator instance.

[Thumb] Set correct initial mapping symbol for big-endian thumb
ClosedPublic

Authored by olista01 on Sep 14 2016, 5:54 AM.

Details

Summary

The initial mapping symbol state is set from the triple, but we only checked for the little-endian thumb triple, so could end up with an ARM mapping symbol for big-endian thumb.

Diff Detail

Repository
rL LLVM

Event Timeline

olista01 updated this revision to Diff 71327.Sep 14 2016, 5:54 AM
olista01 retitled this revision from to [Thumb] Set correct initial mapping symbol for big-endian thumb.
olista01 updated this object.
olista01 added reviewers: rengolin, t.p.northover.
olista01 set the repository for this revision to rL LLVM.
olista01 added a subscriber: llvm-commits.
rengolin accepted this revision.Sep 18 2016, 11:22 AM
rengolin edited edge metadata.

Ouch! Thanks for the fix. LGTM.

--renato

This revision is now accepted and ready to land.Sep 18 2016, 11:22 AM
This revision was automatically updated to reflect the committed changes.