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

Jeff Law law@redhat.com
Fri Jul 25 21:24:00 GMT 2014


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 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, keeping only one SCHED_GROUP_P assignment at the end
>>> of the
>>> function.
>>>
>>> Alexander
>> Thanks for the pointer, I had hurried a bit.
>> Here is the updated patch.
>>
>> Kyrill
>>
>> 2014-07-14  Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>
>>               Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>>
>>       * sched-deps.c (try_group_insn): Generalise macro fusion hook usage
>>       to any two insns.  Update comment.  Rename to
>> sched_macro_fuse_insns.
>>       (sched_analyze_insn): Update use of try_group_insn to
>>       sched_macro_fuse_insns.
>>       * config/i386/i386.c (ix86_macro_fusion_pair_p): Reject 2nd
>> arguments
>>       that are not conditional jumps.
This is fine.  Thanks for your patience.

jeff




More information about the Gcc-patches mailing list