On 29 May 2018 at 19:34, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: > James Greenhalgh wrote: > >> > Add a missing ? to aarch64_get_lane to fix a failure in the testsuite. >> >> > I'd prefer more detail than this for a workaround; which test, why did it >> > start to fail, why is this the right solution, etc. > > It was gcc.target/aarch64/vect_copy_lane_1.c generating: > > test_copy_laneq_f64: > umov x0, v1.d[1] > fmov d0, x0 > ret > > For some reason returning a double uses DImode temporaries, so it's essential > to prefer FP_REGS here and mark the lane copy correctly. > > Wilco >
Hi Wilco, This has probably been reported elsewhere already but I can't find such a report, so sorry for possible duplicate, but this patch is causing ICEs on aarch64 FAIL: gcc.target/aarch64/sve/reduc_1.c -march=armv8.2-a+sve (internal compiler error) FAIL: gcc.target/aarch64/sve/reduc_5.c -march=armv8.2-a+sve (internal compiler error) and also many scan-assembler regressions: http://people.linaro.org/~christophe.lyon/cross-validation/gcc/trunk/260951/report-build-info.html Can you check? Thanks Christophe