This is an archive of the discontinued LLVM Phabricator instance.

[LTO] Fix (wrong) TLS attribute mismatch
ClosedPublic

Authored by davide on May 31 2016, 10:28 PM.

Details

Summary

When we undefine, we also preserve type of symbol so that we get it right in the combined LTO object.

Diff Detail

Event Timeline

davide updated this revision to Diff 59174.May 31 2016, 10:28 PM
davide retitled this revision from to [LTO] Fix TLS attribute mismatch.
davide updated this object.
davide added a reviewer: rafael.
davide retitled this revision from [LTO] Fix TLS attribute mismatch to [LTO] Fix (wrong) TLS attribute mismatch.May 31 2016, 10:28 PM
davide added a reviewer: pcc.
davide added a subscriber: llvm-commits.
rafael accepted this revision.Jun 1 2016, 9:18 AM
rafael edited edge metadata.

LGTM

test/ELF/lto/tls-preserve.ll
12

Simplify, you don't need the arguments for example.

This revision is now accepted and ready to land.Jun 1 2016, 9:18 AM
This revision was automatically updated to reflect the committed changes.