On 03/08/12 19:10, Richard Henderson wrote:
> On 2012-08-02 09:24, Richard Earnshaw wrote:
>> +        /* Extz only supports SImode, but we can coerce the operands
>> +           into that mode.  */
>> +        emit_constant_insn (cond,
>> +                            gen_extzv_t2 (gen_lowpart (mode, target),
>> +                                          gen_lowpart (mode, source),
>> +                                          GEN_INT (i), const0_rtx));
> 
> Didn't you mean gen_lowpart (SImode, ...) ?
> 
> 
> r~
> 

Urm, yes.  Well spotted.

Fixed thusly:

2012-08-04  Richard Earnshaw  <rearn...@arm.com>

        * arm.c (arm_gen_constant): Use SImode when preparing operands for
        gen_extzv_t2.

--- arm.c       (revision 190143)
+++ arm.c       (local)
@@ -2999,8 +2999,8 @@ arm_gen_constant (enum rtx_code code, en
            /* Extz only supports SImode, but we can coerce the operands
               into that mode.  */
            emit_constant_insn (cond,
-                               gen_extzv_t2 (gen_lowpart (mode, target),
-                                             gen_lowpart (mode, source),
+                               gen_extzv_t2 (gen_lowpart (SImode, target),
+                                             gen_lowpart (SImode, source),
                                              GEN_INT (i), const0_rtx));
        }
 

Reply via email to