This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Fix handling of mapping symbols when changing sections
ClosedPublic

Authored by john.brawn on Apr 19 2017, 7:56 AM.

Details

Summary

ChangeSection incorrectly registers LastEMSInfo as belonging to the previous section, not the current section. This happens to work when changing sections using .section, as the previous section is set to the current section before the call to ChangeSection, but not when using .popsection.

Diff Detail

Repository
rL LLVM

Event Timeline

john.brawn created this revision.Apr 19 2017, 7:56 AM
rengolin accepted this revision.Apr 19 2017, 10:05 AM

Good catch, thanks! LGTM.

This revision is now accepted and ready to land.Apr 19 2017, 10:05 AM
This revision was automatically updated to reflect the committed changes.