[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-22 Thread Rainer Orth via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGb1fc966d2e41: [Driver] Support Solaris/amd64 GetTls (authored by ro). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119829/new/ https://reviews.llvm.org/D11

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-19 Thread Rainer Orth via Phabricator via cfe-commits
ro updated this revision to Diff 410083. ro added a comment. Testcase formatting improvements. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119829/new/ https://reviews.llvm.org/D119829 Files: clang/lib/Driver/ToolChains/Solaris.cpp clang/test

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-19 Thread Rainer Orth via Phabricator via cfe-commits
ro added inline comments. Comment at: clang/test/Driver/solaris-ld-sanitizer.c:6 +/// Check sparc-sun-solaris2.11, 32bit +// RUN: %clang %s -### 2>&1 \ +// RUN: --target=sparc-sun-solaris2.11 \ MaskRay wrote: > The first line is now shorter. You can move --ta

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-18 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added inline comments. Comment at: clang/test/Driver/solaris-ld-sanitizer.c:6 +/// Check sparc-sun-solaris2.11, 32bit +// RUN: %clang %s -### 2>&1 \ +// RUN: --target=sparc-sun-solaris2.11 \ The first line is now shorte

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-18 Thread Rainer Orth via Phabricator via cfe-commits
ro updated this revision to Diff 409881. ro marked 2 inline comments as done. ro added a comment. Test fixes as per review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119829/new/ https://reviews.llvm.org/D119829 Files: clang/lib/Driv

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-18 Thread Rainer Orth via Phabricator via cfe-commits
ro marked 2 inline comments as done. ro added inline comments. Comment at: clang/test/Driver/solaris-ld-sanitizer.c:1 +// General tests that the ld -z relax=transtls workaround is only applied +// on Solaris/amd64. Note that we use sysroot to make these tests Mas

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-17 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/Driver/solaris-ld-sanitizer.c:1 +// General tests that the ld -z relax=transtls workaround is only applied +// on Solaris/amd64. Note that we use

[PATCH] D119829: [Driver] Support Solaris/amd64 GetTls

2022-02-15 Thread Rainer Orth via Phabricator via cfe-commits
ro created this revision. ro added a reviewer: MaskRay. ro added a project: clang. Herald added subscribers: fedor.sergeev, jyknight. ro requested review of this revision. This is the driver part of D91605 , a workaround to allow direct calls to `__tls_get_addr` o