On 8 September 2014 11:29, Kyrill Tkachov <kyrylo.tkac...@arm.com> wrote:
> Hi all,
>
> The included testcase currently ICEs at -O0 because vget_lane_f64 is a
> function, so if it's properly called with a constant argument but without
> constant propagation it will not be recognised as constant, causing an ICE.
> This patch changes it to use the macro version directly.
>
> I think there is work being done to fix this issue up as part of a more
> general rework, but until that comes this patch implements the concerned
> intrinsics using the __aarch64_vget_lane* macros like the other lane
> intrinsics around them.
>
> Tested aarch64-none-elf.
>
> Ok for trunk?
>
> Thanks,
> Kyrill
>
> 2014-09-08  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>
>
>         * config/aarch64/arm_neon.h (vmuld_lane_f64): Use macro for getting
>         the lane.
>         (vmuld_laneq_f64): Likewise.
>         (vmuls_lane_f32): Likewise.
>         (vmuls_laneq_f32): Likewise.
>
> 2014-09-08  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>
>
>         * gcc.target/aarch64/simd/vmul_lane_const_lane_1.c: New test.

OK /Marcus

Reply via email to