Re: [PATCH 0/2] backport c++ header fixes to gcc-5-branch

[Richard Biener]

On Tue, Sep 19, 2017 at 5:47 PM, Jack Howarth  wrote:
> On Mon, Sep 18, 2017 at 1:48 PM, Mike Stump  wrote:
>> I was hoping an RM would approve this as it seems just a hair beyond a 
>> normal darwin approval.  I'm fine with this, and it does help darwin.  Other 
>> ports should not care.
>
> Mike,
>  Unless I am misreading this...
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037#c12
>
> wasn't that an approval from Richard Biener for applying the backport
> way back on 9/14?

It was intended as one.

Richard.

>  Jack
>
>>
>> On Sep 18, 2017, at 10:31 AM, Jack Howarth  wrote:
>>>
>>> Pinging for the final gcc 5.5 release.
>>>
>>> On Mon, Aug 7, 2017 at 1:12 AM, Ryan Mounce  wrote:
 Fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037

 Bootstrap now succeeds using Xcode 9 toolchain.

 Tested on macOS 10.13 beta, however same issue reported on macOS 10.12
 with Xcode 9.

 Ryan Mounce (2):
  (header usage fix) remove unused system header includes
  (header usage fix) include c++ headers in system.h

 gcc/ChangeLog| 29 +
 gcc/auto-profile.c   |  6 ++
 gcc/c/c-objc-common.c|  2 --
 gcc/config/sh/sh.c   |  2 +-
 gcc/config/sh/sh_treg_combine.cc |  7 +++
 gcc/cp/error.c   |  2 --
 gcc/diagnostic.c |  2 --
 gcc/fortran/error.c  |  2 --
 gcc/fortran/trans-common.c   |  2 +-
 gcc/genmatch.c   |  1 -
 gcc/graphite-isl-ast-to-gimple.c |  2 +-
 gcc/ipa-icf-gimple.c |  2 +-
 gcc/ipa-icf.c|  2 +-
 gcc/pretty-print.c   |  2 --
 gcc/system.h | 12 
 gcc/toplev.c |  2 --
 16 files changed, 51 insertions(+), 26 deletions(-)

 --
 2.13.2 (Apple Git-90)

>>

Re: [PATCH 0/2] backport c++ header fixes to gcc-5-branch

[Jack Howarth]

On Mon, Sep 18, 2017 at 1:48 PM, Mike Stump  wrote:
> I was hoping an RM would approve this as it seems just a hair beyond a normal 
> darwin approval.  I'm fine with this, and it does help darwin.  Other ports 
> should not care.

Mike,
 Unless I am misreading this...

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037#c12

wasn't that an approval from Richard Biener for applying the backport
way back on 9/14?
 Jack

>
> On Sep 18, 2017, at 10:31 AM, Jack Howarth  wrote:
>>
>> Pinging for the final gcc 5.5 release.
>>
>> On Mon, Aug 7, 2017 at 1:12 AM, Ryan Mounce  wrote:
>>> Fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037
>>>
>>> Bootstrap now succeeds using Xcode 9 toolchain.
>>>
>>> Tested on macOS 10.13 beta, however same issue reported on macOS 10.12
>>> with Xcode 9.
>>>
>>> Ryan Mounce (2):
>>>  (header usage fix) remove unused system header includes
>>>  (header usage fix) include c++ headers in system.h
>>>
>>> gcc/ChangeLog| 29 +
>>> gcc/auto-profile.c   |  6 ++
>>> gcc/c/c-objc-common.c|  2 --
>>> gcc/config/sh/sh.c   |  2 +-
>>> gcc/config/sh/sh_treg_combine.cc |  7 +++
>>> gcc/cp/error.c   |  2 --
>>> gcc/diagnostic.c |  2 --
>>> gcc/fortran/error.c  |  2 --
>>> gcc/fortran/trans-common.c   |  2 +-
>>> gcc/genmatch.c   |  1 -
>>> gcc/graphite-isl-ast-to-gimple.c |  2 +-
>>> gcc/ipa-icf-gimple.c |  2 +-
>>> gcc/ipa-icf.c|  2 +-
>>> gcc/pretty-print.c   |  2 --
>>> gcc/system.h | 12 
>>> gcc/toplev.c |  2 --
>>> 16 files changed, 51 insertions(+), 26 deletions(-)
>>>
>>> --
>>> 2.13.2 (Apple Git-90)
>>>
>

Re: [PATCH 0/2] backport c++ header fixes to gcc-5-branch

[Mike Stump]

I was hoping an RM would approve this as it seems just a hair beyond a normal 
darwin approval.  I'm fine with this, and it does help darwin.  Other ports 
should not care.

On Sep 18, 2017, at 10:31 AM, Jack Howarth  wrote:
> 
> Pinging for the final gcc 5.5 release.
> 
> On Mon, Aug 7, 2017 at 1:12 AM, Ryan Mounce  wrote:
>> Fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037
>> 
>> Bootstrap now succeeds using Xcode 9 toolchain.
>> 
>> Tested on macOS 10.13 beta, however same issue reported on macOS 10.12
>> with Xcode 9.
>> 
>> Ryan Mounce (2):
>>  (header usage fix) remove unused system header includes
>>  (header usage fix) include c++ headers in system.h
>> 
>> gcc/ChangeLog| 29 +
>> gcc/auto-profile.c   |  6 ++
>> gcc/c/c-objc-common.c|  2 --
>> gcc/config/sh/sh.c   |  2 +-
>> gcc/config/sh/sh_treg_combine.cc |  7 +++
>> gcc/cp/error.c   |  2 --
>> gcc/diagnostic.c |  2 --
>> gcc/fortran/error.c  |  2 --
>> gcc/fortran/trans-common.c   |  2 +-
>> gcc/genmatch.c   |  1 -
>> gcc/graphite-isl-ast-to-gimple.c |  2 +-
>> gcc/ipa-icf-gimple.c |  2 +-
>> gcc/ipa-icf.c|  2 +-
>> gcc/pretty-print.c   |  2 --
>> gcc/system.h | 12 
>> gcc/toplev.c |  2 --
>> 16 files changed, 51 insertions(+), 26 deletions(-)
>> 
>> --
>> 2.13.2 (Apple Git-90)
>> 



smime.p7s
Description: S/MIME cryptographic signature

Re: [PATCH 0/2] backport c++ header fixes to gcc-5-branch

[Jack Howarth]

Pinging for the final gcc 5.5 release.

On Mon, Aug 7, 2017 at 1:12 AM, Ryan Mounce  wrote:
> Fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037
>
> Bootstrap now succeeds using Xcode 9 toolchain.
>
> Tested on macOS 10.13 beta, however same issue reported on macOS 10.12
> with Xcode 9.
>
> Ryan Mounce (2):
>   (header usage fix) remove unused system header includes
>   (header usage fix) include c++ headers in system.h
>
>  gcc/ChangeLog| 29 +
>  gcc/auto-profile.c   |  6 ++
>  gcc/c/c-objc-common.c|  2 --
>  gcc/config/sh/sh.c   |  2 +-
>  gcc/config/sh/sh_treg_combine.cc |  7 +++
>  gcc/cp/error.c   |  2 --
>  gcc/diagnostic.c |  2 --
>  gcc/fortran/error.c  |  2 --
>  gcc/fortran/trans-common.c   |  2 +-
>  gcc/genmatch.c   |  1 -
>  gcc/graphite-isl-ast-to-gimple.c |  2 +-
>  gcc/ipa-icf-gimple.c |  2 +-
>  gcc/ipa-icf.c|  2 +-
>  gcc/pretty-print.c   |  2 --
>  gcc/system.h | 12 
>  gcc/toplev.c |  2 --
>  16 files changed, 51 insertions(+), 26 deletions(-)
>
> --
> 2.13.2 (Apple Git-90)
>

Re: [PATCH 0/2] backport c++ header fixes to gcc-5-branch

[Jack Howarth]

Szabolcs,
 Can you help get these back ports into gcc-5-branch?

https://gcc.gnu.org/ml/gcc-patches/2017-08/msg00478.html
https://gcc.gnu.org/ml/gcc-patches/2017-08/msg00477.html

Thanks in advance.
 Jack

On Mon, Aug 7, 2017 at 1:12 AM, Ryan Mounce  wrote:
> Fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81037
>
> Bootstrap now succeeds using Xcode 9 toolchain.
>
> Tested on macOS 10.13 beta, however same issue reported on macOS 10.12
> with Xcode 9.
>
> Ryan Mounce (2):
>   (header usage fix) remove unused system header includes
>   (header usage fix) include c++ headers in system.h
>
>  gcc/ChangeLog| 29 +
>  gcc/auto-profile.c   |  6 ++
>  gcc/c/c-objc-common.c|  2 --
>  gcc/config/sh/sh.c   |  2 +-
>  gcc/config/sh/sh_treg_combine.cc |  7 +++
>  gcc/cp/error.c   |  2 --
>  gcc/diagnostic.c |  2 --
>  gcc/fortran/error.c  |  2 --
>  gcc/fortran/trans-common.c   |  2 +-
>  gcc/genmatch.c   |  1 -
>  gcc/graphite-isl-ast-to-gimple.c |  2 +-
>  gcc/ipa-icf-gimple.c |  2 +-
>  gcc/ipa-icf.c|  2 +-
>  gcc/pretty-print.c   |  2 --
>  gcc/system.h | 12 
>  gcc/toplev.c |  2 --
>  16 files changed, 51 insertions(+), 26 deletions(-)
>
> --
> 2.13.2 (Apple Git-90)
>

[PATCH 0/2] backport c++ header fixes to gcc-5-branch

[Ryan Mounce]

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

Bootstrap now succeeds using Xcode 9 toolchain.

Tested on macOS 10.13 beta, however same issue reported on macOS 10.12
with Xcode 9.

Ryan Mounce (2):
  (header usage fix) remove unused system header includes
  (header usage fix) include c++ headers in system.h

 gcc/ChangeLog| 29 +
 gcc/auto-profile.c   |  6 ++
 gcc/c/c-objc-common.c|  2 --
 gcc/config/sh/sh.c   |  2 +-
 gcc/config/sh/sh_treg_combine.cc |  7 +++
 gcc/cp/error.c   |  2 --
 gcc/diagnostic.c |  2 --
 gcc/fortran/error.c  |  2 --
 gcc/fortran/trans-common.c   |  2 +-
 gcc/genmatch.c   |  1 -
 gcc/graphite-isl-ast-to-gimple.c |  2 +-
 gcc/ipa-icf-gimple.c |  2 +-
 gcc/ipa-icf.c|  2 +-
 gcc/pretty-print.c   |  2 --
 gcc/system.h | 12 
 gcc/toplev.c |  2 --
 16 files changed, 51 insertions(+), 26 deletions(-)

-- 
2.13.2 (Apple Git-90)