On 31 August 2012 17:59, Richard Henderson <r...@redhat.com> wrote:
> On 2012-08-31 07:25, Christophe Lyon wrote:
>> +  offset = gen_rtx_CONST_INT (VOIDmode, location);
>
> Never call gen_rtx_CONST_INT directly.  Use GEN_INT.
>
>
Here is an updated patch with that small change.
For the record, there are quite a few existing calls to
gen_rtx_CONST_INT, maybe a cleanup pass is needed?

Thanks,

Christophe.

2012-09-03  Christophe Lyon  <christophe.l...@linaro.org>

        gcc/
        * config/arm/arm.c (arm_evpc_neon_vext): New
        function.
        (arm_expand_vec_perm_const_1): Add call to
        arm_evpc_neon_vext.


        gcc/testsuite/
        * gcc.target/arm/neon-vext.c
        gcc.target/arm/neon-vext-execute.c:
        New tests.

Attachment: gcc-vec-permute-vext.patch
Description: Binary data

Reply via email to