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


Reply via email to