[PATCH] Remove -Wabi from libstdc++ build options

[PATCH] Remove -Wabi from libstdc++ build options

[Bernd Edlinger]

[-- Attachment #1: Type: text/plain, Size: 521 bytes --]

Hi,

the -Wabi option prints a warning as follows:

cc1plus: warning: -Wabi won't warn about anything [-Wabi]
cc1plus: note: -Wabi warns about differences from the most up-to-date 
ABI, which is also used by default
cc1plus: note: use e.g. -Wabi=11 to warn about changes from GCC 7

This happens many times while building libstdc++, and as the warning
explains, it is good for nothing, so this patch removes it.


Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
Is it OK for trunk?


Thanks
Bernd.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch-remove-wabi.diff --]
[-- Type: text/x-patch; name="patch-remove-wabi.diff", Size: 1018 bytes --]

2018-06-30  Bernd Edlinger  <bernd.edlinger@hotmail.de>

	* acinclude.m4 (WARN_FLAGS): Remove -Wabi.
	* configure: Regenerated.

Index: libstdc++-v3/acinclude.m4
===================================================================
--- libstdc++-v3/acinclude.m4	(revision 262271)
+++ libstdc++-v3/acinclude.m4	(working copy)
@@ -733,7 +733,7 @@
   # OPTIMIZE_CXXFLAGS = -O3 -fstrict-aliasing -fvtable-gc
   AC_SUBST(OPTIMIZE_CXXFLAGS)
 
-  WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual -Wabi'
+  WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
   AC_SUBST(WARN_FLAGS)
 ])
 
Index: libstdc++-v3/configure
===================================================================
--- libstdc++-v3/configure	(revision 262271)
+++ libstdc++-v3/configure	(working copy)
@@ -81846,7 +81846,7 @@
   # OPTIMIZE_CXXFLAGS = -O3 -fstrict-aliasing -fvtable-gc
 
 
-  WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual -Wabi'
+  WARN_FLAGS='-Wall -Wextra -Wwrite-strings -Wcast-qual'
 
 
 

Re: [PATCH] Remove -Wabi from libstdc++ build options

[Jonathan Wakely]

On 30/06/18 20:48 +0000, Bernd Edlinger wrote:
>Hi,
>
>the -Wabi option prints a warning as follows:
>
>cc1plus: warning: -Wabi won't warn about anything [-Wabi]
>cc1plus: note: -Wabi warns about differences from the most up-to-date
>ABI, which is also used by default
>cc1plus: note: use e.g. -Wabi=11 to warn about changes from GCC 7
>
>This happens many times while building libstdc++, and as the warning
>explains, it is good for nothing, so this patch removes it.
>
>
>Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
>Is it OK for trunk?

No, I don't think we want to simply remove it.

Maybe https://gcc.gnu.org/ml/gcc/2018-06/msg00276.html instead?