PATCH: eliminate -Wc++compat-warning from dominance.c

Ben Elliston bje@au1.ibm.com
Fri Jan 23 04:59:00 GMT 2009


> Are you happy with the definition of BITMAP_FREE given above?

BTW, here is the proper patch and ChangeLog entry.  Tested with a
bootstrap on x86_64-linux.  Okay for the trunk?


2009-01-23  Ben Elliston  <bje@au.ibm.com>

        * bitmap.h (BITMAP_FREE): Eliminateq `implicit conversion from
        void *' warning from -Wc++-compat.
        * Makefile.in (dominance.o-warn): Remove.

Index: bitmap.h
===================================================================
--- bitmap.h    (revision 143582)
+++ bitmap.h    (working copy)
@@ -194,8 +194,8 @@
 #define BITMAP_GGC_ALLOC() bitmap_gc_alloc ()
 
 /* Do any cleanup needed on a bitmap when it is no longer used.  */
-#define BITMAP_FREE(BITMAP)                    \
-       ((void)(bitmap_obstack_free (BITMAP), (BITMAP) = NULL))
+#define BITMAP_FREE(BITMAP) \
+       ((void) (bitmap_obstack_free ((bitmap) BITMAP), (BITMAP) = (bitmap) NULL))
 
 /* Iterator for bitmaps.  */
 
Index: Makefile.in
===================================================================
--- Makefile.in (revision 143582)
+++ Makefile.in (working copy)
@@ -179,8 +179,6 @@
 SYSCALLS.c.X-warn = -Wno-strict-prototypes -Wno-error
 # dfp.c contains alias violations
 dfp.o-warn = -Wno-error
-# dominance.c contains a -Wc++compat warning.
-dominance.o-warn = -Wno-error
 # mips-tfile.c contains -Wcast-qual warnings.
 mips-tfile.o-warn = -Wno-error
 




More information about the Gcc-patches mailing list