Skip to content

Commit 86dc26e

Browse files
committed
[clang] Don't pass sanitizers' rtlibs to linker with -r
If clang is invoked with `-r` (relocatable linking) then we shouldn't construct sanitizer arguments and then pass sanitizer runtime libs to the linker GCC also skips runtime linking if `-r` is also here
1 parent 69ff853 commit 86dc26e

File tree

2 files changed

+18
-1
lines changed

2 files changed

+18
-1
lines changed

clang/lib/Driver/SanitizerArgs.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1159,7 +1159,8 @@ SanitizerArgs::SanitizerArgs(const ToolChain &TC,
11591159

11601160
LinkRuntimes =
11611161
Args.hasFlag(options::OPT_fsanitize_link_runtime,
1162-
options::OPT_fno_sanitize_link_runtime, LinkRuntimes);
1162+
options::OPT_fno_sanitize_link_runtime, LinkRuntimes) &&
1163+
!Args.hasArg(options::OPT_r);
11631164

11641165
// Parse -link-cxx-sanitizer flag.
11651166
LinkCXXRuntimes = D.CCCIsCXX();

clang/test/Driver/sanitizer-ld.c

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1369,3 +1369,19 @@
13691369
// CHECK-DSO-SHARED-HWASAN-AARCH64-LINUX-NOT: "-lresolv"
13701370
// CHECK-DSO-SHARED-HWASAN-AARCH64-LINUX-NOT: "--export-dynamic"
13711371
// CHECK-DSO-SHARED-HWASAN-AARCH64-LINUX-NOT: "--dynamic-list"
1372+
1373+
// RUN: %clang -fsanitize=address,undefined -r -### %s 2>&1 \
1374+
// RUN: --target=x86_64-unknown-linux -fuse-ld=ld \
1375+
// RUN: -resource-dir=%S/Inputs/resource_dir \
1376+
// RUN: --sysroot=%S/Inputs/basic_linux_tree \
1377+
// RUN: | %{filecheck} --check-prefix=CHECK-RELOCATABLE-LINK-ASAN-UBSAN-RTLIB
1378+
//
1379+
// CHECK-RELOCATABLE-LINK-ASAN-UBSAN-RTLIB-NOT: "{{.*}}(asan|ubsan){{.*}}"
1380+
1381+
// RUN: %clang -fsanitize=thread -r -### %s 2>&1 \
1382+
// RUN: --target=x86_64-unknown-linux -fuse-ld=ld \
1383+
// RUN: -resource-dir=%S/Inputs/resource_dir \
1384+
// RUN: --sysroot=%S/Inputs/basic_linux_tree \
1385+
// RUN: | %{filecheck} --check-prefix=CHECK-RELOCATABLE-LINK-TSAN-RTLIB
1386+
//
1387+
// CHECK-RELOCATABLE-LINK-TSAN-RTLIB-NOT: "{{.*}}tsan{{.*}}"

0 commit comments

Comments
 (0)