On 01/13/2017 04:50 AM, Alan Modra wrote:
Rather than using unspecs in doloop insns to stop combine creating
these insns, use legitimate_combined_insn.

I'm not sure why the original patch implementing
legitimate_combined_insn did not store the result of recog to the insn
but it seems good to me, and would allow the recog call in
ix86_legitimate_combined_insn to be omitted.  (I tested that too, not
shown here.)

Bootstrapped and regression tested powerpc64le-linux, powerpc64-linux,
x86_64-linux.  OK for mainline?

        PR target/72749
        * combine.c (recog_for_combine_1): Set INSN_CODE before calling
        target legitimate_combined_insn.
        * config/rs6000/rs6000.c (TARGET_LEGITIMATE_COMBINED_INSN): Define.
        (rs6000_legitimate_combined_insn): New function.
        * config/rs6000/rs6000.md (UNSPEC_DOLOOP): Delete, and remove
        all uses.
        (ctr<mode>_internal3): Rename from *ctr<mode>_internal5.
        (ctr<mode>_internal4): Rename from *ctr<mode>_internal6.
        (ctr<mode>_internal1, ctr<mode>_internal2): Remove '*' from name.
OK on the combine.c changes.  ppc maintainers own the rest.


Reply via email to