Hi Tamar,

On 15/01/19 17:05, Tamar Christina wrote:
Hi All,

We are a bit inconsistent when it comes to lane index endianness on Arm,
we don't seem to always stick to the expected GCC vector extensions index
endianness, for these tests since they are modelled as UNSPEC anyway just
keep the indexes in Arm NEON order.

There are other intrinsics that require an update, but for now these will
fix the new ones.

Bootstrapped Regtested on arm-none-Linux-gnueabihf and no issues.
Cross compiled on armeb-none-eabi and regtested and no issues.
Verified example by hand with execution tests and no issues.

Ok for trunk?


Ok.
We should aim to make the lane-flipping consistent in the future.

Thanks,
Kyrill

Thanks,
Tamar

gcc/ChangeLog:

2019-01-15  Tamar Christina  <tamar.christ...@arm.com>

        * config/arm/arm-protos.h (neon_vcmla_lane_prepare_operands): Remove 
patternmode.
        * config/arm/arm.c (neon_vcmla_lane_prepare_operands): Likewise.
        * config/arm/neon.md (neon_vcmla_lane<rot><mode>, 
neon_vcmla_laneq<rot><mode>,
        neon_vcmlaq_lane<rot><mode>): Remove endianness conversion.

--

Reply via email to