Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-25 Thread Jeff Law
On 07/24/14 03:11, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00958.html Kyrill On 14/07/14 11:01, Kyrill Tkachov wrote: On 11/07/14 14:20, Alexander Monakov wrote: On Fri, 11 Jul 2014, Kyrill Tkachov wrote: On 10/07/14 22:53, Maxim Kuvyrkov wrote: The patch l

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-24 Thread Maxim Kuvyrkov
On Jul 24, 2014, at 10:11 AM, Kyrill Tkachov wrote: > Ping. > https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00958.html > > Kyrill > > On 14/07/14 11:01, Kyrill Tkachov wrote: >> On 11/07/14 14:20, Alexander Monakov wrote: >>> On Fri, 11 Jul 2014, Kyrill Tkachov wrote: On 10/07/14 22:53, M

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-24 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00958.html Kyrill On 14/07/14 11:01, Kyrill Tkachov wrote: On 11/07/14 14:20, Alexander Monakov wrote: On Fri, 11 Jul 2014, Kyrill Tkachov wrote: On 10/07/14 22:53, Maxim Kuvyrkov wrote: The patch looks good to me, but some cleanup suggesti

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-14 Thread Kyrill Tkachov
On 11/07/14 14:20, Alexander Monakov wrote: On Fri, 11 Jul 2014, Kyrill Tkachov wrote: On 10/07/14 22:53, Maxim Kuvyrkov wrote: The patch looks good to me, but some cleanup suggestions below. Thanks, here's an updated patch. How's this? You need to remove 'if (targetm. ...) SCHED_GROUP_P (in

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-11 Thread Kyrill Tkachov
On 11/07/14 14:20, Alexander Monakov wrote: On Fri, 11 Jul 2014, Kyrill Tkachov wrote: On 10/07/14 22:53, Maxim Kuvyrkov wrote: The patch looks good to me, but some cleanup suggestions below. Thanks, here's an updated patch. How's this? You need to remove 'if (targetm. ...) SCHED_GROUP_P (in

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-11 Thread Alexander Monakov
On Fri, 11 Jul 2014, Kyrill Tkachov wrote: > > On 10/07/14 22:53, Maxim Kuvyrkov wrote: > > The patch looks good to me, but some cleanup suggestions below. > > Thanks, here's an updated patch. > How's this? You need to remove 'if (targetm. ...) SCHED_GROUP_P (insn) = 1;' from the first if branch

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-11 Thread Kyrill Tkachov
On 10/07/14 22:53, Maxim Kuvyrkov wrote: On Jul 10, 2014, at 8:00 PM, Kyrill Tkachov wrote: On 30/06/14 21:39, Jeff Law wrote: On 06/27/14 02:29, Kyrill Tkachov wrote: Hi all, This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work on more than just compares and conditiona

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-10 Thread Maxim Kuvyrkov
On Jul 10, 2014, at 8:00 PM, Kyrill Tkachov wrote: > > On 30/06/14 21:39, Jeff Law wrote: >> On 06/27/14 02:29, Kyrill Tkachov wrote: >>> Hi all, >>> >>> This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work >>> on more than just >>> compares and conditional branches for whic

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-07-10 Thread Kyrill Tkachov
On 30/06/14 21:39, Jeff Law wrote: On 06/27/14 02:29, Kyrill Tkachov wrote: Hi all, This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work on more than just compares and conditional branches for which it was initially designed for (for x86). There are some instructions in ar

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-06-30 Thread Jeff Law
On 06/27/14 02:29, Kyrill Tkachov wrote: Hi all, This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work on more than just compares and conditional branches for which it was initially designed for (for x86). There are some instructions in arm and aarch64 that can be fused toget

Re: [PATCH][sched-deps] Generalise usage of macro fusion to work on any two insns

2014-06-30 Thread Kyrill Tkachov
On 27/06/14 09:29, Kyrill Tkachov wrote: Hi all, This patch generalises the TARGET_MACRO_FUSION_PAIR_P hook usage to work on more than just compares and conditional branches for which it was initially designed for (for x86). There are some instructions in arm and aarch64 that can be fused toge