Re: [RFC combine] PR48308 - Fix issue with missing(?) LOG_LINKS

[Ramana Radhakrishnan <ramana dot radhakrishnan at linaro dot org>]

On 23 January 2012 09:02, Eric Botcazou <ebotcazou@adacore.com> wrote:
>> Do you mean something like this instead ? I'm testing this - Ok if no
>> regressions ?
>
> OK with an appropriate ChangeLog.

Or not :( We really shouldn't use SUBST there as it only deals with
rtx *  It appeared better to invent a new SUBST for links given that
the type of this expression is struct insn_link rather than rtx * (Doh
!)  I did think about hacking up a local flag but somehow like this
version better.

It's survived a bootstrap and testrun with no regressions for
c,c++,fortran,ada on one of the power7 machines for
powerpc64-unknown-linux-gnu in the compile farm. I'll put it through
it's paces for trunk (arm-linux-gnueabi / x86_64) and 4.6 (modulo API
changes for alloc_insn_link) if you think this looks better.

cheers
Ramana



2012-01-23  Ramana Radhakrishnan  <ramana.radhakrishnan@linaro.org>

        PR rtl-optimization/48308
        * combine.c (do_SUBST_LINK): Declare and define.
        (SUBST_LINK): New.
        (try_combine): Handle LOG_LINKS for dummy i1 case.

>
> --
> Eric Botcazou

Attachment: pr48308-combine.txt
Description: Text document