Skip to content

Commit

Permalink
wrappers: Stop specifying -fuse-cxa-atexit
Browse files Browse the repository at this point in the history
Since clang 9.0.0 (34ec6929a1fb33716d0035dc0f1f164d7568b4dc
in llvm-project), it's no longer necessary to set this to be able
to use __cxa_thread_atexit (for TLS destructors). This brings the
behaviour closer to GCC, and reduces the amount of non-default
options used.
  • Loading branch information
mstorsjo committed Dec 20, 2019
1 parent 7f2171d commit bc6df72
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion wrappers/clang-target-wrapper.c
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,6 @@ int _tmain(int argc, TCHAR* argv[]) {
exec_argv[arg++] = _T("-rtlib=compiler-rt");
exec_argv[arg++] = _T("-stdlib=libc++");
exec_argv[arg++] = _T("-fuse-ld=lld");
exec_argv[arg++] = _T("-fuse-cxa-atexit");
exec_argv[arg++] = _T("-Qunused-arguments");

for (int i = 1; i < argc; i++)
Expand Down
1 change: 0 additions & 1 deletion wrappers/clang-target-wrapper.sh
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ FLAGS="$FLAGS -target $TARGET"
FLAGS="$FLAGS -rtlib=compiler-rt"
FLAGS="$FLAGS -stdlib=libc++"
FLAGS="$FLAGS -fuse-ld=lld"
FLAGS="$FLAGS -fuse-cxa-atexit"
FLAGS="$FLAGS -Qunused-arguments"

$CCACHE "$CLANG" $FLAGS "$@"

0 comments on commit bc6df72

Please sign in to comment.