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

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

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 kyrylo.tkac...@arm.com 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

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

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 kyrylo.tkac...@arm.com 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

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 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

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

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 kyrylo.tkac...@arm.com 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

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

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

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

2014-06-27 Thread Kyrill Tkachov
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 together when they're back to back in the