This is an archive of the discontinued LLVM Phabricator instance.

Haiku: Enable thread-local storag, disable PIE / enable PIC by default
ClosedPublic

Authored by brad on Sep 1 2023, 4:14 PM.

Details

Summary

Derived from https://reviews.llvm.org/D49481, but added tests and simplified the diff a bit; isPIEDefault can just be removed. Also looking through their upstream patches I noticed PIC is also enabled.

Diff Detail

Event Timeline

brad created this revision.Sep 1 2023, 4:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 1 2023, 4:14 PM
brad requested review of this revision.Sep 1 2023, 4:14 PM
brad updated this revision to Diff 555564.Sep 1 2023, 11:26 PM

x86_64 test for TLS.

brad updated this revision to Diff 555604.Sep 2 2023, 3:02 PM
brad retitled this revision from Haiku: Enable thread-local storage and disable PIE by default to Haiku: Enable thread-local storag, disable PIE / enable PIC by default.
brad edited the summary of this revision. (Show Details)
nielx added a comment.Sep 2 2023, 10:55 PM

Can confirm this is correct for Haiku.
For PIC by default, see our GCC haiku.h.
These changes have also been part of our patches for clang.

nielx accepted this revision.Sep 2 2023, 10:56 PM

Looks good to me!

This revision is now accepted and ready to land.Sep 2 2023, 10:56 PM
This revision was landed with ongoing or failed builds.Sep 2 2023, 11:03 PM
This revision was automatically updated to reflect the committed changes.