[compiler-rt] [compiler-rt] Switch LLD specific tests to a more precise option (PR #69781)
Martin Storsjö via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 20 14:23:36 PDT 2023
https://github.com/mstorsjo created https://github.com/llvm/llvm-project/pull/69781
Prefer the new flag -lld-allow-duplicate-weak (which was added recently in a67ae8c0fd301a11e2a058e8035304cfc70a3e91), over the old -lldmingw.
The -lldmingw option enables a number of different behaviours, while this test only is interested in one aspect of them.
This should allow making -lldmingw more strict with not enabling MSVC specific behaviours like inferring lib directories automatically from the environment, as being pursued in https://reviews.llvm.org/D144084.
>From 18dc1f531f1dc3707f53dd1fb011166b3ac1cab8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?= <martin at martin.st>
Date: Sun, 1 Oct 2023 00:02:16 +0300
Subject: [PATCH] [compiler-rt] Switch LLD specific tests to a more precise
option
Prefer the new flag -lld-allow-duplicate-weak (which was added
recently in a67ae8c0fd301a11e2a058e8035304cfc70a3e91), over the old
-lldmingw.
The -lldmingw option enables a number of different behaviours,
while this test only is interested in one aspect of them.
This should allow making -lldmingw more strict with not enabling
MSVC specific behaviours like inferring lib directories automatically
from the environment, as being pursued in https://reviews.llvm.org/D144084.
---
compiler-rt/test/profile/Windows/coverage-weak-lld.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/compiler-rt/test/profile/Windows/coverage-weak-lld.cpp b/compiler-rt/test/profile/Windows/coverage-weak-lld.cpp
index 338be8e979c564e..e239d809640ff94 100644
--- a/compiler-rt/test/profile/Windows/coverage-weak-lld.cpp
+++ b/compiler-rt/test/profile/Windows/coverage-weak-lld.cpp
@@ -14,7 +14,7 @@
// RUN: llvm-profdata show %t.profraw --all-functions | FileCheck %s --check-prefix=PROFILE1
/// link.exe does not support weak overridding weak.
-// RUN: %clang_profgen -fcoverage-mapping -fuse-ld=lld -Wl,-lldmingw,-opt:ref %t0.o %t2.o -o %t2
+// RUN: %clang_profgen -fcoverage-mapping -fuse-ld=lld -Wl,-lld-allow-duplicate-weak,-opt:ref %t0.o %t2.o -o %t2
// RUN: env LLVM_PROFILE_FILE=%t.profraw %run %t2 | FileCheck %s --check-prefix=CHECK2
// RUN: llvm-profdata show %t.profraw --all-functions | FileCheck %s --check-prefix=PROFILE2
@@ -30,7 +30,7 @@
// RUN: env LLVM_PROFILE_FILE=%t.profraw %run %t1 | FileCheck %s --check-prefix=CHECK1
// RUN: llvm-profdata show %t.profraw --all-functions | FileCheck %s --check-prefix=PROFILE1
-// RUN: %clang_profgen -fcoverage-mapping -fuse-ld=lld -Wl,-lldmingw,-opt:ref %t0.o %t2.o -o %t2
+// RUN: %clang_profgen -fcoverage-mapping -fuse-ld=lld -Wl,-lld-allow-duplicate-weak,-opt:ref %t0.o %t2.o -o %t2
// RUN: env LLVM_PROFILE_FILE=%t.profraw %run %t2 | FileCheck %s --check-prefix=CHECK2
// RUN: llvm-profdata show %t.profraw --all-functions | FileCheck %s --check-prefix=PROFILE2
More information about the llvm-commits
mailing list