Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-12-08 Thread Jeff Law
On 12/08/14 14:12, Ilya Enkovich wrote: 2014-12-09 0:00 GMT+03:00 Jeff Law : On 12/08/14 04:00, Ilya Enkovich wrote: 2014-11-26 Ilya Enkovich PR bootstrap/63995 * tree-chkp.c (chkp_make_static_bounds): Share bounds var between nodes sharing assembler name. gc

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-12-08 Thread Ilya Enkovich
2014-12-09 0:00 GMT+03:00 Jeff Law : > On 12/08/14 04:00, Ilya Enkovich wrote: >>> >>> 2014-11-26 Ilya Enkovich >>> >>> PR bootstrap/63995 >>> * tree-chkp.c (chkp_make_static_bounds): Share bounds var >>> between nodes sharing assembler name. >>> >>> gcc/testsuite >>>

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-12-08 Thread Jeff Law
On 12/08/14 04:00, Ilya Enkovich wrote: 2014-11-26 Ilya Enkovich PR bootstrap/63995 * tree-chkp.c (chkp_make_static_bounds): Share bounds var between nodes sharing assembler name. gcc/testsuite 2014-11-26 Ilya Enkovich PR bootstrap/63995 * g+

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-12-08 Thread Ilya Enkovich
Ping 2014-11-26 15:35 GMT+03:00 Ilya Enkovich : > On 25 Nov 15:03, Ilya Enkovich wrote: >> 2014-11-25 14:11 GMT+03:00 Richard Biener : >> > On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich >> > wrote: >> > >> > Ok, then it's get_for_asmname (). That said - the above loops look >> > bogus to me.

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-26 Thread Ilya Enkovich
2014-11-26 19:07 GMT+03:00 Jan Hubicka : >> On 25 Nov 15:03, Ilya Enkovich wrote: >> > 2014-11-25 14:11 GMT+03:00 Richard Biener : >> > > On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich >> > > wrote: >> > > >> > > Ok, then it's get_for_asmname (). That said - the above loops look >> > > bogus to

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-26 Thread Jan Hubicka
> On 25 Nov 15:03, Ilya Enkovich wrote: > > 2014-11-25 14:11 GMT+03:00 Richard Biener : > > > On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich > > > wrote: > > > > > > Ok, then it's get_for_asmname (). That said - the above loops look > > > bogus to me. Honza - any better ideas? > > > > get_for

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-26 Thread Ilya Enkovich
On 25 Nov 15:03, Ilya Enkovich wrote: > 2014-11-25 14:11 GMT+03:00 Richard Biener : > > On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich > > wrote: > > > > Ok, then it's get_for_asmname (). That said - the above loops look > > bogus to me. Honza - any better ideas? > > get_for_asmname () retur

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Ilya Enkovich
2014-11-25 14:11 GMT+03:00 Richard Biener : > On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich > wrote: >> 2014-11-25 12:43 GMT+03:00 Richard Biener : >>> On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich >>> wrote: Hi, This patch partly fixes PR bootstrap/63995 by avoiding duplicati

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 11:19 AM, Ilya Enkovich wrote: > 2014-11-25 12:43 GMT+03:00 Richard Biener : >> On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich >> wrote: >>> Hi, >>> >>> This patch partly fixes PR bootstrap/63995 by avoiding duplicating static >>> bounds vars. With this fix bootstrap st

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Ilya Enkovich
2014-11-25 12:43 GMT+03:00 Richard Biener : > On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich wrote: >> Hi, >> >> This patch partly fixes PR bootstrap/63995 by avoiding duplicating static >> bounds vars. With this fix bootstrap still fails at stage 2 and 3 >> comparison. >> >> Bootstrapped and c

Re: [PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Richard Biener
On Tue, Nov 25, 2014 at 9:45 AM, Ilya Enkovich wrote: > Hi, > > This patch partly fixes PR bootstrap/63995 by avoiding duplicating static > bounds vars. With this fix bootstrap still fails at stage 2 and 3 comparison. > > Bootstrapped and checked on x86_64-unknown-linux-gnu. OK for trunk? > > T

[PATCH, PR63995, CHKP] Use single static bounds var for varpool nodes sharing asm name

2014-11-25 Thread Ilya Enkovich
Hi, This patch partly fixes PR bootstrap/63995 by avoiding duplicating static bounds vars. With this fix bootstrap still fails at stage 2 and 3 comparison. Bootstrapped and checked on x86_64-unknown-linux-gnu. OK for trunk? Thanks, Ilya -- gcc/ 2014-11-25 Ilya Enkovich PR bootstr