Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-02-03 Thread Jeff Law
On 01/27/15 07:27, Jakub Jelinek wrote: Hi! I've grepped for BUILT_IN_.*_CHKP in the sources and we actually need far fewer enum values than the 1204 that are being defined. This patch requires builtins.def to say explicitly (by using DEF_*BUILTIN_CHKP macro instead of corresponding DEF_*BUILTI

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Jeff Law
On 01/28/15 12:24, Richard Biener wrote: It should be the STABS and/or affected target maintainers job to get this fixed for them. Richard, Even if the STABS continuations are fixed, it requires fixing it in previous releases of GCC, deploying the solution and achieving adoption. The curren

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Jeff Law
On 01/28/15 12:57, Jakub Jelinek wrote: On Wed, Jan 28, 2015 at 12:51:24PM -0700, Jeff Law wrote: On 01/28/15 12:24, Richard Biener wrote: It should be the STABS and/or affected target maintainers job to get this fixed for them. Richard, Even if the STABS continuations are fixed, it requir

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Jakub Jelinek
On Wed, Jan 28, 2015 at 12:51:24PM -0700, Jeff Law wrote: > On 01/28/15 12:24, Richard Biener wrote: > >>>It should be the STABS and/or affected target maintainers job to get > >>this fixed > >>>for them. > >> > >>Richard, > >> > >>Even if the STABS continuations are fixed, it requires fixing it in

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Richard Biener
On January 28, 2015 4:23:05 PM CET, David Edelsohn wrote: >On Wed, Jan 28, 2015 at 6:45 AM, Richard Biener > wrote: >> On Wed, Jan 28, 2015 at 12:24 PM, Jakub Jelinek >wrote: >>> On Wed, Jan 28, 2015 at 12:15:40PM +0100, Richard Biener wrote: > Note, patch successfully bootstrapped/regtested

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread David Edelsohn
On Wed, Jan 28, 2015 at 6:45 AM, Richard Biener wrote: > On Wed, Jan 28, 2015 at 12:24 PM, Jakub Jelinek wrote: >> On Wed, Jan 28, 2015 at 12:15:40PM +0100, Richard Biener wrote: >>> > Note, patch successfully bootstrapped/regtested on x86_64-linux and >>> > i686-linux, and David said that on AIX

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Richard Biener
On Wed, Jan 28, 2015 at 12:24 PM, Jakub Jelinek wrote: > On Wed, Jan 28, 2015 at 12:15:40PM +0100, Richard Biener wrote: >> > Note, patch successfully bootstrapped/regtested on x86_64-linux and >> > i686-linux, and David said that on AIX it passed stage1 cc1 linking. >> > >> > Ok for trunk? >> >>

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Jakub Jelinek
On Wed, Jan 28, 2015 at 12:15:40PM +0100, Richard Biener wrote: > > Note, patch successfully bootstrapped/regtested on x86_64-linux and > > i686-linux, and David said that on AIX it passed stage1 cc1 linking. > > > > Ok for trunk? > > Is the stabs issue meanwhile fixed at least on trunk? AFAIK no

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-28 Thread Richard Biener
On Tue, Jan 27, 2015 at 11:42 PM, Jakub Jelinek wrote: > On Tue, Jan 27, 2015 at 06:04:53PM +0300, Ilya Enkovich wrote: >> 2015-01-27 17:27 GMT+03:00 Jakub Jelinek : >> > I've grepped for BUILT_IN_.*_CHKP in the sources and we actually need >> > far fewer enum values than the 1204 that are being d

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-27 Thread Jakub Jelinek
On Tue, Jan 27, 2015 at 06:04:53PM +0300, Ilya Enkovich wrote: > 2015-01-27 17:27 GMT+03:00 Jakub Jelinek : > > I've grepped for BUILT_IN_.*_CHKP in the sources and we actually need > > far fewer enum values than the 1204 that are being defined. > > > > This patch requires builtins.def to say expli

Re: [RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-27 Thread Ilya Enkovich
2015-01-27 17:27 GMT+03:00 Jakub Jelinek : > Hi! > > I've grepped for BUILT_IN_.*_CHKP in the sources and we actually need > far fewer enum values than the 1204 that are being defined. > > This patch requires builtins.def to say explicitly (by using > DEF_*BUILTIN_CHKP macro instead of correspondin

[RFC PATCH] Avoid most of the BUILT_IN_*_CHKP enum values

2015-01-27 Thread Jakub Jelinek
Hi! I've grepped for BUILT_IN_.*_CHKP in the sources and we actually need far fewer enum values than the 1204 that are being defined. This patch requires builtins.def to say explicitly (by using DEF_*BUILTIN_CHKP macro instead of corresponding DEF_*BUILTIN) which ones need that, for all the other