[PATCH] D154397: [Driver] Default to -ftls-model=initial-exec on SerenityOS

2023-07-05 Thread Daniel Bertalan via Phabricator via cfe-commits
BertalanD added a comment. Hi MaskRay! Your blog post (alongside the Drepper TLS doc) proved to be a very helpful resource for our implementation of the dynamic TLS interface. I currently have a SerenityOS PR open that implements `__tls_get_a

[PATCH] D154397: [Driver] Default to -ftls-model=initial-exec on SerenityOS

2023-07-05 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. I don't think this belongs to the upstream, either. Note: musl's TLS implementation is very clean and may be a good reference. https://maskray.me/blog/2021-02-14-all-about-thread-local-storage > glibc ld.so reserves some space in static TLS blocks and allows dlopen on

[PATCH] D154397: [Driver] Default to -ftls-model=initial-exec on SerenityOS

2023-07-03 Thread Andrew Kaster via Phabricator via cfe-commits
ADKaster abandoned this revision. ADKaster added a comment. In D154397#4470094 , @jrtc27 wrote: > This is pretty dodgy, I don't think it belongs upstream Fair enough, the original patch had "probably don't upstream this" in the commit description. @Bert

[PATCH] D154397: [Driver] Default to -ftls-model=initial-exec on SerenityOS

2023-07-03 Thread Jessica Clarke via Phabricator via cfe-commits
jrtc27 added a comment. This is pretty dodgy, I don't think it belongs upstream Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154397/new/ https://reviews.llvm.org/D154397 ___ cfe-commits mailing list cfe

[PATCH] D154397: [Driver] Default to -ftls-model=initial-exec on SerenityOS

2023-07-03 Thread Andrew Kaster via Phabricator via cfe-commits
ADKaster created this revision. Herald added a project: All. ADKaster requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. Force Clang use the initial-exec TLS model instead of the default local-exec when building code for Serenity. Thi