Re: [PATCH][ARM/AArch64] PR 68088: Fix RTL checking ICE due to subregs inside accumulator forwarding check

2015-11-09 Thread Nikolai Bozhenov
On 11/06/2015 08:16 PM, Kyrill Tkachov wrote: On 06/11/15 17:09, Kyrill Tkachov wrote: On 06/11/15 17:07, Nikolai Bozhenov wrote: On 11/06/2015 04:46 PM, Ramana Radhakrishnan wrote: Hi! I faced the same issue but I had somewhat different RTL for the consumer: (insn 20 15 21 2

Re: [PATCH][ARM/AArch64] PR 68088: Fix RTL checking ICE due to subregs inside accumulator forwarding check

2015-11-06 Thread Nikolai Bozhenov
On 11/06/2015 08:09 PM, Kyrill Tkachov wrote: On 06/11/15 17:07, Nikolai Bozhenov wrote: On 11/06/2015 04:46 PM, Ramana Radhakrishnan wrote: Hi! I faced the same issue but I had somewhat different RTL for the consumer: (insn 20 15 21 2 (set (reg/i:SI 0 r0) (minus:SI

Re: Re: [PATCH][ARM/AArch64] PR 68088: Fix RTL checking ICE due to subregs inside accumulator forwarding check

2015-11-06 Thread Nikolai Bozhenov
On 11/06/2015 04:46 PM, Ramana Radhakrishnan wrote: Hi! I faced the same issue but I had somewhat different RTL for the consumer: (insn 20 15 21 2 (set (reg/i:SI 0 r0) (minus:SI (subreg:SI (reg:DI 117) 4) (mult:SI (reg:SI 123) (reg:SI 114)

Re: [PATCH][ARM/AArch64] PR 68088: Fix RTL checking ICE due to subregs inside accumulator forwarding check

2015-11-06 Thread Nikolai Bozhenov
On 10/28/2015 01:07 PM, Kyrill Tkachov wrote: Hi all, This RTL checking error occurs on aarch64 in aarch_accumulator_forwarding when processing an msubsi insn with subregs: (insn 15 14 16 3 (set (reg/v:SI 78 [ i ]) (minus:SI (subreg:SI (reg/v:DI 76 [ aul ]) 0) (mult:SI (sub

Re: [sched] Dump dependency graph to a dot file

2015-11-05 Thread Nikolai Bozhenov
On 11/05/2015 12:18 PM, Kyrill Tkachov wrote: Hi Nikolai, On 05/11/15 08:29, Nikolai Bozhenov wrote: Hi! The attached patch adds a procedure to dump the scheduler's dependency graph into a dot file. The patch has been bootstrapped and regtested on x86_64. Please commit if it is OK for

Re: [ping] Change behavior of -fsched-verbose option

2015-11-05 Thread Nikolai Bozhenov
On 11/05/2015 12:26 PM, Bernd Schmidt wrote: On 11/05/2015 09:11 AM, Nikolai Bozhenov wrote: Please commit if it is OK for trunk. Hmm, do you have a copyright assignment on file? Yes I do have a copyright assignment for all past and future changes to GCC (RT:828836). Thanks, Nikolai

[sched] Dump dependency graph to a dot file

2015-11-05 Thread Nikolai Bozhenov
Hi! The attached patch adds a procedure to dump the scheduler's dependency graph into a dot file. The patch has been bootstrapped and regtested on x86_64. Please commit if it is OK for trunk. Thanks, Nikolai 2015-11-04 Nikolai Bozhenov * sched-int.h (dump_rgn_dependencies_dot): De

[ping] Change behavior of -fsched-verbose option

2015-11-05 Thread Nikolai Bozhenov
Please commit if it is OK for trunk. Thanks, Nikolai On 10/22/2015 06:38 PM, Nikolai Bozhenov wrote: Hi! Currently -fsched-verbose option redirects debugging dumps to stderr if there is no dump_file for the current pass. It would be fine if there were the only scheduling pass. But for example

Re: Change behavior of -fsched-verbose option

2015-10-22 Thread Nikolai Bozhenov
On 10/22/2015 06:56 PM, Bernd Schmidt wrote: On 10/22/2015 05:38 PM, Nikolai Bozhenov wrote: Currently -fsched-verbose option redirects debugging dumps to stderr if there is no dump_file for the current pass. It would be fine if there were the only scheduling pass. But for example for AArch64

Change behavior of -fsched-verbose option

2015-10-22 Thread Nikolai Bozhenov
e old behavior may be reproduced with options -fsched-verbose=7 -fdump-rtl-sched1 -fdump-rtl-{sched_fusion,sched2}=stderr if it is really necessary. The patch has been bootstrapped and regtested on x86_64. Thanks, Nikolai 2015-10-22 Nikolai Bozhenov * haifa-sched.c (setup_sched_dump): Don'

Re: Fix prototype for print_insn in rtl.h

2015-10-20 Thread Nikolai Bozhenov
On 10/15/2015 09:42 PM, Trevor Saunders wrote: Sorry, a little late to the party.. but why is print_insn even in rtl.h? it seems that sched-vis.c is the only thing that uses it... Andrew I'm going to use it in the scheduler... but then wouldn't something like sched-int.h make more sense? On t

Re: Fix prototype for print_insn in rtl.h

2015-10-15 Thread Nikolai Bozhenov
On 10/15/2015 07:28 PM, Andrew MacLeod wrote: On 10/13/2015 11:32 AM, Jeff Law wrote: On 10/13/2015 02:21 AM, Nikolai Bozhenov wrote: 2015-10-13 Nikolai Bozhenov * gcc/rtl.h (print_insn): fix prototype Installed on the trunk after bootstrap & regression test. jeff Sorry, a li

Re: Fix prototype for print_insn in rtl.h

2015-10-13 Thread Nikolai Bozhenov
On 10/13/2015 03:22 PM, Jeff Law wrote: On 10/13/2015 02:21 AM, Nikolai Bozhenov wrote: Currently prototype for print_insn in rtl.h doesn't match it's definition in sched-vis.c The patch fixes this mismatch. I'll run this through the usual bootstrap & regression testing befor

Fix prototype for print_insn in rtl.h

2015-10-13 Thread Nikolai Bozhenov
Currently prototype for print_insn in rtl.h doesn't match it's definition in sched-vis.c The patch fixes this mismatch. Thanks, Nikolai 2015-10-13 Nikolai Bozhenov * gcc/rtl.h (print_insn): fix prototype diff --git a/gcc/rtl.h b/gcc/rtl.h index a592a1e..d6edc71 100644 --- a

Re: [RFC, PATCH] Disable -fprofile-use related optimizations if corresponding .gcda file not found.

2015-10-08 Thread Nikolai Bozhenov
On 10/07/2015 08:07 PM, pins...@gmail.com wrote: Yes. But I am saying some one could do -fprofile-use -frename-registers and expect rename registers to stay on even if there is no profile. That's true, we shouldn't disable any options that were explicitly requested by the user. Why not fi

[PATCH, libstdc++, testsuite] Remove redundant -save-temps options

2015-08-04 Thread Nikolai Bozhenov
Hi, the attached patch removes redundant -save-temps options from some libstdc++ tests, since the option is not needed in dg-do-compile/scan-assembler tests. Thanks, Nikolai 2015-08-04 Nikolai Bozhenov * testsuite/20_util/enable_shared_from_this/cons/constexpr.cc: Remove redundant -save