Move OVERRIDE/FINAL from gcc/coretypes.h to include/ansidecl.h
authorPedro Alves <palves@redhat.com>
Fri, 14 Oct 2016 16:03:16 +0000 (17:03 +0100)
committerPedro Alves <palves@redhat.com>
Fri, 14 Oct 2016 16:03:16 +0000 (17:03 +0100)
So that GDB and other projects that share the top level can use them.

Bootstrapped with all default languages + jit on x86-64 Fedora 23.

gcc/ChangeLog:
2016-10-14  Pedro Alves  <palves@redhat.com>

* coretypes.h (OVERRIDE, FINAL): Delete, moved to
include/ansidecl.h.

include/ChangeLog:
2016-10-14  Pedro Alves  <palves@redhat.com>

* ansidecl.h (GCC_FINAL): Delete.
(OVERRIDE, FINAL): New, moved from gcc/coretypes.h.

include/ChangeLog
include/ansidecl.h

index 6f681308ebb8cf813b208d473d2c966eb0668ab3..bdff2dd9947879f0aabf4dc21e1af0d53bf95ad0 100644 (file)
@@ -1,3 +1,8 @@
+2016-10-14  Pedro Alves  <palves@redhat.com>
+
+       * ansidecl.h (GCC_FINAL): Delete.
+       (OVERRIDE, FINAL): New, moved from gcc/coretypes.h.
+
 2016-10-14  Claudiu Zissulescu  <claziss@synopsys.com>
 
        * opcode/arc.h (ARC_OPCODE_ARCV2): New define.
index 18b782121fad2cd0a32abfedfea30b5bd6e58a93..7dd7baaeddfa24008090b0861fef63163ac2da6b 100644 (file)
@@ -313,13 +313,29 @@ So instead we use the macro below and test it against specific values.  */
 #define ENUM_BITFIELD(TYPE) unsigned int
 #endif
 
-    /* This is used to mark a class or virtual function as final.  */
-#if __cplusplus >= 201103L
-#define GCC_FINAL final
-#elif GCC_VERSION >= 4007
-#define GCC_FINAL __final
+/* C++11 adds the ability to add "override" after an implementation of a
+   virtual function in a subclass, to:
+     (A) document that this is an override of a virtual function
+     (B) allow the compiler to issue a warning if it isn't (e.g. a mismatch
+         of the type signature).
+
+   Similarly, it allows us to add a "final" to indicate that no subclass
+   may subsequently override the vfunc.
+
+   Provide OVERRIDE and FINAL as macros, allowing us to get these benefits
+   when compiling with C++11 support, but without requiring C++11.
+
+   For gcc, use "-std=c++11" to enable C++11 support; gcc 6 onwards enables
+   this by default (actually GNU++14).  */
+
+#if __cplusplus >= 201103
+/* C++11 claims to be available: use it: */
+#define OVERRIDE override
+#define FINAL final
 #else
-#define GCC_FINAL
+/* No C++11 support; leave the macros empty: */
+#define OVERRIDE
+#define FINAL
 #endif
 
 #ifdef __cplusplus