Re: Fix PR57268

2013-06-03 Thread Jeff Law
On 06/01/2013 06:55 AM, Jakub Jelinek wrote: On Sat, Jun 01, 2013 at 10:11:24AM +0200, Jakub Jelinek wrote: On Sat, Jun 01, 2013 at 08:39:58AM +0400, Dinar Temirbulatov wrote: I am investigating the problem. I'd guess you don't want to flush on DEBUG_INSN_Ps, because then you'd flush differen

Re: Fix PR57268

2013-06-03 Thread Dinar Temirbulatov
> I'd guess you don't want to flush on DEBUG_INSN_Ps, because then you'd flush > differently between -g and -g0. So perhaps something like: yes, If I skipped to flush all DEBUG_INSN_Ps, then dependence lists are the same under -g0 and -g. I bootstrapped Jakub's change on x86_64-linux with no erro

Re: Fix PR57268

2013-06-01 Thread Jakub Jelinek
On Sat, Jun 01, 2013 at 10:11:24AM +0200, Jakub Jelinek wrote: > On Sat, Jun 01, 2013 at 08:39:58AM +0400, Dinar Temirbulatov wrote: > > I am investigating the problem. > > I'd guess you don't want to flush on DEBUG_INSN_Ps, because then you'd flush > differently between -g and -g0. So perhaps so

Re: Fix PR57268

2013-06-01 Thread Jakub Jelinek
On Sat, Jun 01, 2013 at 08:39:58AM +0400, Dinar Temirbulatov wrote: > oh, This is my mistake I should have bootstrap the compiler. That is a requirement for most of commits, see http://gcc.gnu.org/contribute.html for the exact rules. I am investigating the problem. I'd guess you don't want to fl

Re: Fix PR57268

2013-05-31 Thread Dinar Temirbulatov
Hi, oh, This is my mistake I should have bootstrap the compiler. I am investigating the problem. thanks, Dinar. On Sat, Jun 1, 2013 at 7:50 AM, H.J. Lu wrote: > On Wed, May 29, 2013 at 9:36 AM, Dinar Temirbulatov > wrote: >> Here is the corrected version of change. Also, I think, I nee

Re: Fix PR57268

2013-05-31 Thread H.J. Lu
On Wed, May 29, 2013 at 9:36 AM, Dinar Temirbulatov wrote: > Here is the corrected version of change. Also, I think, I need > write-after-approval access to commit the change. > thanks, Dinar, > > PR rtl-optimization/57268 > * sched-deps.c (sched_analyze_2): Flush d

Re: Fix PR57268

2013-05-29 Thread Jeff Law
On 05/29/2013 10:36 AM, Dinar Temirbulatov wrote: Here is the corrected version of change. Also, I think, I need write-after-approval access to commit the change. thanks, Dinar, PR rtl-optimization/57268 * sched-deps.c (sched_analyze_2): Flush dependence lists

Re: Fix PR57268

2013-05-29 Thread Maxim Kuvyrkov
On 30/05/2013, at 4:36 AM, Dinar Temirbulatov wrote: > Here is the corrected version of change. Also, I think, I need > write-after-approval access to commit the change. >thanks, Dinar, > >PR rtl-optimization/57268 >* sched-deps.c (sched_analyze_2): Flush dependenc

Re: Fix PR57268

2013-05-29 Thread Dinar Temirbulatov
Here is the corrected version of change. Also, I think, I need write-after-approval access to commit the change. thanks, Dinar, PR rtl-optimization/57268 * sched-deps.c (sched_analyze_2): Flush dependence lists if the sum of the read and write lists exceeds

Re: Fix PR57268

2013-05-29 Thread Dinar Temirbulatov
* sched-deps.c (sched_analyze_2): Flush dependence lists if the sum of the read and write lists exceeds MAX_PENDING_LIST_LENGTH. On Wed, May 29, 2013 at 6:49 PM, Jeff Law wrote: > On 05/29/2013 06:52 AM, Steven Bosscher wrote: >> >> Hello, >> >> On Wed, May 29, 2013 at 2:01 PM, Dinar Temi

Re: Fix PR57268

2013-05-29 Thread Jeff Law
On 05/29/2013 06:52 AM, Steven Bosscher wrote: Hello, On Wed, May 29, 2013 at 2:01 PM, Dinar Temirbulatov wrote: Hi, I noticed that the scheduler created long dependence list about ~9000 elements long and slowed compilation time for about an hour. Attached patch flushes the dependence list is c

Re: Fix PR57268

2013-05-29 Thread Steven Bosscher
Hello, On Wed, May 29, 2013 at 2:01 PM, Dinar Temirbulatov wrote: > Hi, > I noticed that the scheduler created long dependence list about ~9000 > elements long and slowed compilation time for about an hour. Attached > patch flushes the dependence list is case of it is longer than > MAX_PENDING_LIS