* [RFC PATCH] configure: Do not add --warn-common to the linker flags anymore
@ 2021-05-25 11:11 Thomas Huth
2021-05-25 23:22 ` Richard Henderson
0 siblings, 1 reply; 2+ messages in thread
From: Thomas Huth @ 2021-05-25 11:11 UTC (permalink / raw)
To: qemu-devel, Paolo Bonzini
We are compiling with -fno-common since commit 4c288acbd6 ("configure:
Always build with -fno-common"), so --warn-common (which had been added
in commit 49237acdb725e in 2008 already) should not be necessary anymore
nowadays.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
Marked as RFC since I'm not 100% sure whether I'm missing here something...
but IMHO the --warn-common does not buy us anything if we also compile
with -fno-common...
configure | 7 -------
1 file changed, 7 deletions(-)
diff --git a/configure b/configure
index 86a8a7e44f..99b9216af3 100755
--- a/configure
+++ b/configure
@@ -5383,13 +5383,6 @@ fi
##########################################
-# Exclude --warn-common with TSan to suppress warnings from the TSan libraries.
-if test "$solaris" = "no" && test "$tsan" = "no"; then
- if $ld --version 2>/dev/null | grep "GNU ld" >/dev/null 2>/dev/null ; then
- QEMU_LDFLAGS="-Wl,--warn-common $QEMU_LDFLAGS"
- fi
-fi
-
# Use ASLR, no-SEH and DEP if available
if test "$mingw32" = "yes" ; then
flags="--no-seh --nxcompat"
--
2.27.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [RFC PATCH] configure: Do not add --warn-common to the linker flags anymore
2021-05-25 11:11 [RFC PATCH] configure: Do not add --warn-common to the linker flags anymore Thomas Huth
@ 2021-05-25 23:22 ` Richard Henderson
0 siblings, 0 replies; 2+ messages in thread
From: Richard Henderson @ 2021-05-25 23:22 UTC (permalink / raw)
To: Thomas Huth, qemu-devel, Paolo Bonzini
On 5/25/21 4:11 AM, Thomas Huth wrote:
> We are compiling with -fno-common since commit 4c288acbd6 ("configure:
> Always build with -fno-common"), so --warn-common (which had been added
> in commit 49237acdb725e in 2008 already) should not be necessary anymore
> nowadays.
>
> Signed-off-by: Thomas Huth<thuth@redhat.com>
> ---
> Marked as RFC since I'm not 100% sure whether I'm missing here something...
> but IMHO the --warn-common does not buy us anything if we also compile
> with -fno-common...
I agree.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-05-25 23:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-25 11:11 [RFC PATCH] configure: Do not add --warn-common to the linker flags anymore Thomas Huth
2021-05-25 23:22 ` Richard Henderson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).