Christophe Lyon <christophe.l...@linaro.org> writes: > 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
Thanks for the heads-up. Looks like they're all SVE, so I'll take this. Richard
