[x86,PATCH] Simple fix for Atom LEA splitting.

Uros Bizjak ubizjak@gmail.com
Mon Sep 16 14:26:00 GMT 2013


On Mon, Sep 16, 2013 at 2:50 PM, Yuri Rumyantsev <ysrumyan@gmail.com> wrote:
> Hi All,
>
> Here is 1-line fix which improves Atom performance by better distance
> estimation. We got ~20% speedup on one of bench from eembc2.0 with
> this fix.
>
> Bootstrapping and regression testing were successful for x86-64.
>
> Is it OK for trunk?
>
> ChangeLog:
>
> 2013-09-16  Yuri Rumyantsev  <ysrumyan@gmail.com>
>
> * config/i386/i386.c (distance_agu_use_in_bb) : Proper initialization
> of 'prev' var to get better distance estimation.

Please rewrite this part to:

  basic_block bb = NULL;
  rtx next = start;
  rtx prev = NULL;

  if (start != NULL_RTX)
    {
      bb = BLOCK_FOR_INSN (start);
      if (start != BB_HEAD (bb))
    prev = insn;
    }

A comment would be also nice here. It took me quite some time to
decipher the code.

OK with these changes.

Thanks,
Uros.



More information about the Gcc-patches mailing list