[Bug preprocessor/51776] New: fixincludes hacks around a C++ deficiency

[Bug preprocessor/51776] New: fixincludes hacks around a C++ deficiency

[bkorb at gnu dot org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

             Bug #: 51776
           Summary: fixincludes hacks around a C++ deficiency
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: preprocessor
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: bkorb@gnu.org


a new fixinclude hack is about to be added to paper over the fact that
g++ -std=c++11 claims full compliance, but is slightly deficient.
Once that is fixed, this new hack should be removed.  It will be
approximately thus:

+/*
+ * g++ -std=c++11 defines __cplusplus to 201103L, but does not
+ * properly support [[noreturn]].
+ */
+fix = {
+    hackname  = cdef_cplusplus;
+    files     = sys/cdefs.h;
+    select    = '\[\[noreturn\]\]';
+    c_fix     = format;
+    c_fix_arg = "__attribute__((__noreturn__))";
+    test_text = "#define _Noreturn     [[noreturn]]";
+};

CF: bug #51705 comment #25

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

[andreast at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

--- Comment #1 from Andreas Tobler <andreast at gcc dot gnu.org> 2012-01-11 16:23:57 UTC ---
Author: andreast
Date: Wed Jan 11 16:23:48 2012
New Revision: 183096

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=183096
Log:
2012-01-11  Bruce Korb <bkorb@gnu.org>
        Steven G. Kargl  <kargl@gcc.gnu.org>
        Andreas Tobler  <andreast@fgznet.ch>

    PR bootstrap/57105
    PR preprocessor/51776
    * inclhack.def (cdef_cplusplus): Add a replacement for [[noreturn]].
    * fixincl.x: Regenerate.
    * tests/base/sys/cdefs.h: Update.

    * genfixes: Remove the 'Ver.' from the version check.


Modified:
    trunk/fixincludes/ChangeLog
    trunk/fixincludes/fixincl.x
    trunk/fixincludes/genfixes
    trunk/fixincludes/inclhack.def
    trunk/fixincludes/tests/base/sys/cdefs.h

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

[redi at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |53528

--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> 2012-05-30 13:33:18 UTC ---
adding the "Support C++11 generalized attributes" PR as a dependency so Bruce
gets prodded when it changes state

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

[bkorb at gnu dot org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

--- Comment #3 from bkorb at gnu dot org 2012-10-08 17:46:04 UTC ---
(In reply to comment #2)
> adding ... PR as a dependency so Bruce gets prodded when it changes state

Bruce has now been prodded. :)  I will fix this in 3 weeks or so.

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

[skannan at redhat dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

Shakthi Kannan <skannan at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |skannan at redhat dot com

--- Comment #4 from Shakthi Kannan <skannan at redhat dot com> ---
The depends bug 53528 "Support C++11 generalized attributes" has been closed,
and applied to trunk (4.8).

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

[bkorb at gnu dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

Bruce Korb <bkorb at gnu dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Bruce Korb <bkorb at gnu dot org> ---
This hack is no longer in inclhack.def.

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

[egallager at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776

Eric Gallager <egallager at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |egallager at gcc dot gnu.org

--- Comment #6 from Eric Gallager <egallager at gcc dot gnu.org> ---
(In reply to Bruce Korb from comment #0)
> CF: bug #51705 comment #25

(In reply to Andreas Tobler from comment #1)
> 	PR bootstrap/57105

OK so there are two similar-looking bug numbers here; I'm assuming the first
one is the correct one, and the second was a typo?