Re: [PATCH] Fix PR81921

2017-08-29 Thread Richard Biener
On Mon, 28 Aug 2017, Joseph Myers wrote: > On Mon, 28 Aug 2017, Joseph Myers wrote: > > > On Sat, 26 Aug 2017, Richard Biener wrote: > > > > > On August 26, 2017 12:51:57 AM GMT+02:00, Joseph Myers > > > wrote: > > > >I'm seeing a build failure for s390x-linux-gnu that looks like it could > >

Re: [PATCH] Fix PR81921

2017-08-28 Thread Joseph Myers
On Mon, 28 Aug 2017, Joseph Myers wrote: > On Sat, 26 Aug 2017, Richard Biener wrote: > > > On August 26, 2017 12:51:57 AM GMT+02:00, Joseph Myers > > wrote: > > >I'm seeing a build failure for s390x-linux-gnu that looks like it could > > >be > > >related to this change (build was OK at r25133

Re: [PATCH] Fix PR81921

2017-08-28 Thread Joseph Myers
On Sat, 26 Aug 2017, Richard Biener wrote: > On August 26, 2017 12:51:57 AM GMT+02:00, Joseph Myers > wrote: > >I'm seeing a build failure for s390x-linux-gnu that looks like it could > >be > >related to this change (build was OK at r251332, failed at r251358). > > Can you please open a bug? C

Re: [PATCH] Fix PR81921

2017-08-25 Thread Richard Biener
On August 26, 2017 12:51:57 AM GMT+02:00, Joseph Myers wrote: >I'm seeing a build failure for s390x-linux-gnu that looks like it could >be >related to this change (build was OK at r251332, failed at r251358). Can you please open a bug? Can you confirm it fails the same way before the patch if

Re: [PATCH] Fix PR81921

2017-08-25 Thread Joseph Myers
I'm seeing a build failure for s390x-linux-gnu that looks like it could be related to this change (build was OK at r251332, failed at r251358). https://sourceware.org/ml/libc-testresults/2017-q3/msg00329.html In file included from /scratch/jmyers/glibc-bot/src/gcc/libitm/libitm_i.h:39:0,

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Wed, 23 Aug 2017, Uros Bizjak wrote: > On Wed, Aug 23, 2017 at 12:18 PM, Richard Biener wrote: > > On Wed, 23 Aug 2017, Uros Bizjak wrote: > > > >> On Wed, Aug 23, 2017 at 11:55 AM, Richard Biener wrote: > >> > On Wed, 23 Aug 2017, Uros Bizjak wrote: > >> > > >> >> On Wed, Aug 23, 2017 at 11:

Re: [PATCH] Fix PR81921

2017-08-23 Thread Uros Bizjak
On Wed, Aug 23, 2017 at 12:18 PM, Richard Biener wrote: > On Wed, 23 Aug 2017, Uros Bizjak wrote: > >> On Wed, Aug 23, 2017 at 11:55 AM, Richard Biener wrote: >> > On Wed, 23 Aug 2017, Uros Bizjak wrote: >> > >> >> On Wed, Aug 23, 2017 at 11:40 AM, Uros Bizjak wrote: >> >> > On Wed, Aug 23, 2017

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Wed, 23 Aug 2017, Uros Bizjak wrote: > On Wed, Aug 23, 2017 at 11:55 AM, Richard Biener wrote: > > On Wed, 23 Aug 2017, Uros Bizjak wrote: > > > >> On Wed, Aug 23, 2017 at 11:40 AM, Uros Bizjak wrote: > >> > On Wed, Aug 23, 2017 at 11:22 AM, Richard Biener > >> > wrote: > >> >>> > /* If th

Re: [PATCH] Fix PR81921

2017-08-23 Thread Uros Bizjak
On Wed, Aug 23, 2017 at 11:55 AM, Richard Biener wrote: > On Wed, 23 Aug 2017, Uros Bizjak wrote: > >> On Wed, Aug 23, 2017 at 11:40 AM, Uros Bizjak wrote: >> > On Wed, Aug 23, 2017 at 11:22 AM, Richard Biener wrote: >> >>> > /* If the machine does not have a case insn that compares the bounds,

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Wed, 23 Aug 2017, Uros Bizjak wrote: > On Wed, Aug 23, 2017 at 11:40 AM, Uros Bizjak wrote: > > On Wed, Aug 23, 2017 at 11:22 AM, Richard Biener wrote: > >>> > /* If the machine does not have a case insn that compares the bounds, > >>> > Index: gcc/config/i386/i386.c > >>> >

Re: [PATCH] Fix PR81921

2017-08-23 Thread Uros Bizjak
On Wed, Aug 23, 2017 at 11:40 AM, Uros Bizjak wrote: > On Wed, Aug 23, 2017 at 11:22 AM, Richard Biener wrote: >>> > /* If the machine does not have a case insn that compares the bounds, >>> > Index: gcc/config/i386/i386.c >>> > ===

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Wed, 23 Aug 2017, Uros Bizjak wrote: > On Wed, Aug 23, 2017 at 11:22 AM, Richard Biener wrote: > >> > /* If the machine does not have a case insn that compares the bounds, > >> > Index: gcc/config/i386/i386.c > >> > === > >> > --

Re: [PATCH] Fix PR81921

2017-08-23 Thread Uros Bizjak
On Wed, Aug 23, 2017 at 11:22 AM, Richard Biener wrote: >> > /* If the machine does not have a case insn that compares the bounds, >> > Index: gcc/config/i386/i386.c >> > === >> > --- gcc/config/i386/i386.c (revision 251275) >>

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Wed, 23 Aug 2017, Uros Bizjak wrote: > On Wed, Aug 23, 2017 at 10:04 AM, Richard Biener wrote: > > On Wed, 23 Aug 2017, Richard Biener wrote: > > > >> On Tue, 22 Aug 2017, Richard Biener wrote: > >> > >> > On August 22, 2017 6:38:55 PM GMT+02:00, "Richard Earnshaw (lists)" > >> > wrote: > >>

Re: [PATCH] Fix PR81921

2017-08-23 Thread Uros Bizjak
On Wed, Aug 23, 2017 at 10:04 AM, Richard Biener wrote: > On Wed, 23 Aug 2017, Richard Biener wrote: > >> On Tue, 22 Aug 2017, Richard Biener wrote: >> >> > On August 22, 2017 6:38:55 PM GMT+02:00, "Richard Earnshaw (lists)" >> > wrote: >> > >On 22/08/17 13:55, Kyrill Tkachov wrote: >> > >> Hi R

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Wed, 23 Aug 2017, Richard Biener wrote: > On Tue, 22 Aug 2017, Richard Biener wrote: > > > On August 22, 2017 6:38:55 PM GMT+02:00, "Richard Earnshaw (lists)" > > wrote: > > >On 22/08/17 13:55, Kyrill Tkachov wrote: > > >> Hi Richard, > > >> [roping in more aarch64 maintainers] > > >> > > >

Re: [PATCH] Fix PR81921

2017-08-23 Thread Richard Biener
On Tue, 22 Aug 2017, Richard Biener wrote: > On August 22, 2017 6:38:55 PM GMT+02:00, "Richard Earnshaw (lists)" > wrote: > >On 22/08/17 13:55, Kyrill Tkachov wrote: > >> Hi Richard, > >> [roping in more aarch64 maintainers] > >> > >> On 22/08/17 13:27, Richard Biener wrote: > >>> On Tue, 22 Au

Re: [PATCH] Fix PR81921

2017-08-22 Thread Richard Biener
On August 22, 2017 6:38:55 PM GMT+02:00, "Richard Earnshaw (lists)" wrote: >On 22/08/17 13:55, Kyrill Tkachov wrote: >> Hi Richard, >> [roping in more aarch64 maintainers] >> >> On 22/08/17 13:27, Richard Biener wrote: >>> On Tue, 22 Aug 2017, Uros Bizjak wrote: >>> On Tue, Aug 22, 2017 at

Re: [PATCH] Fix PR81921

2017-08-22 Thread Richard Earnshaw (lists)
On 22/08/17 13:55, Kyrill Tkachov wrote: > Hi Richard, > [roping in more aarch64 maintainers] > > On 22/08/17 13:27, Richard Biener wrote: >> On Tue, 22 Aug 2017, Uros Bizjak wrote: >> >>> On Tue, Aug 22, 2017 at 12:15 PM, Richard Biener >>> wrote: The following patch fixes PR81921 (and LTO

Re: [PATCH] Fix PR81921

2017-08-22 Thread Segher Boessenkool
On Tue, Aug 22, 2017 at 02:27:28PM +0200, Richard Biener wrote: > Thanks. Of course this was copied by other targets (and the x86 > one maybe from the default). So the following is an extended patch. > > Ok for the rs6000 and aarch64 bits? Sure, rs6000 part is fine. Thanks! Segher >

Re: [PATCH] Fix PR81921

2017-08-22 Thread Kyrill Tkachov
Hi Richard, [roping in more aarch64 maintainers] On 22/08/17 13:27, Richard Biener wrote: On Tue, 22 Aug 2017, Uros Bizjak wrote: On Tue, Aug 22, 2017 at 12:15 PM, Richard Biener wrote: The following patch fixes PR81921 (and LTO build of libgo) which I ran into when trying to enable free-lan

Re: [PATCH] Fix PR81921

2017-08-22 Thread Richard Biener
On Tue, 22 Aug 2017, Uros Bizjak wrote: > On Tue, Aug 22, 2017 at 12:15 PM, Richard Biener wrote: > > > > The following patch fixes PR81921 (and LTO build of libgo) which I ran > > into when trying to enable free-lang-data for non-LTO compiles. > > > > free-lang-data forces a DECL_FUNCTION_SPECIF

Re: [PATCH] Fix PR81921

2017-08-22 Thread Uros Bizjak
On Tue, Aug 22, 2017 at 12:15 PM, Richard Biener wrote: > > The following patch fixes PR81921 (and LTO build of libgo) which I ran > into when trying to enable free-lang-data for non-LTO compiles. > > free-lang-data forces a DECL_FUNCTION_SPECIFIC_TARGET for all functions > so we have them ending

[PATCH] Fix PR81921

2017-08-22 Thread Richard Biener
The following patch fixes PR81921 (and LTO build of libgo) which I ran into when trying to enable free-lang-data for non-LTO compiles. free-lang-data forces a DECL_FUNCTION_SPECIFIC_TARGET for all functions so we have them ending up with target_option_default_node eventually which is something ix