qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] configure: use -Wwombat to test whether gcc rec


From: Igor Mitsyanko
Subject: Re: [Qemu-devel] [PATCH] configure: use -Wwombat to test whether gcc recognizes -Wno-wombat
Date: Sun, 28 Oct 2012 19:03:33 +0400
User-agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:13.0) Gecko/20120614 Thunderbird/13.0.1

On 10/28/2012 1:19 AM, Peter Maydell wrote:
gcc will silently accept unrecognized -Wno-wombat warning suppression
options (it only mentions them if it has to print a compiler warning
for some other reason). Since we already run a check for whether gcc
recognizes the warning options we use, we can easily make this use
the positive sense of the option when checking for support for the
suppression option. This doesn't have any effect except that it avoids
gcc emitting extra messages about unrecognized command line options
when it is printing other warning messages.

Signed-off-by: Peter Maydell <address@hidden>
---
  configure | 6 +++++-
  1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/configure b/configure
index 37f712d..4c2ba06 100755
--- a/configure
+++ b/configure
@@ -1169,7 +1169,11 @@ cat > $TMPC << EOF
  int main(void) { return 0; }
  EOF
  for flag in $gcc_flags; do
-    if compile_prog "-Werror $flag" "" ; then
+    # Use the positive sense of the flag when testing for -Wno-wombat
+    # support (gcc will happily accept the -Wno- form of unknown
+    # warning options).
+    optflag="$(echo $flag | sed -e 's/^-Wno-/-W/')"
+    if compile_prog "-Werror $optflag" "" ; then
        QEMU_CFLAGS="$QEMU_CFLAGS $flag"
      fi
  done


Reviewed-by: Igor Mitsyanko <address@hidden>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]