RE: [Patch] improve bfin conditional move support

["Henderson, Stuart" <Stuart dot Henderson at analog dot com>]

e.g.
> -  operands[1] = bfin_gen_compare (operands[1], SImode);
> +  operands[1] = bfin_gen_compare (operands[1], <MODE>mode);
?

Updated patch attached.

Thanks,
Stu

2011-03-29  Stuart Henderson  <stuart.henderson@analog.com>

    From Bernd Schmidt:
    * config/bfin/bfin.md (MOVCC): New mode_macro.
    (mov<mode>cc_insn1, mov<mode>cc_insn2, mov<mode>cc): Renamed from
    movsicc_insn1, movsicc_insn2 and movsicc and macroized.  Remove
    comments from generated assembly.

-----Original Message-----
From: Richard Henderson [mailto:rth@redhat.com]
Sent: 30 March 2011 18:29
To: Henderson, Stuart
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [Patch] improve bfin conditional move support

On 03/29/2011 05:57 AM, Henderson, Stuart wrote:
> -  operands[1] = bfin_gen_compare (operands[1], SImode);
> +  operands[1] = bfin_gen_compare (operands[1], GET_MODE (operands[0]));

FWIW, you can use "<MODE>mode" to get the proper value
without having to read it from the operand.


r~

Attachment: upstream.patch
Description: upstream.patch