Christophe Lyon <christophe.l...@linaro.org> writes: > On Mon, 30 Sep 2019 at 18:48, Richard Sandiford <richard.sandif...@arm.com> > wrote: > > Richard Sandiford <richard.sandif...@arm.com> writes: > > [This follows on from: > > https://gcc.gnu.org/ml/gcc-patches/2019-09/msg00778.html > > https://gcc.gnu.org/ml/gcc-patches/2019-09/msg01456.html] > > > > With the function ABI stuff, we can now support shrink-wrapping of > > non-leaf vector PCS functions. This is particularly useful if the > > vector PCS function calls an ordinary function on an error path, > > since we can then keep the extra saves and restores specific to > > that path too. > > > > Tested on aarch64-linux-gnu. OK to install? > > Now self-approved :-), applied as r276340. > > > Hi Richard, > > As you may have noticed from gcc-testresults, the new test simd-abi-9.c fails > with -mabi=ilp32.
Thanks for the heads up. The reason for the failure is that on ILP32 targets, there's an extra UXTW instruction to extend the incoming pointer before the load. It doesn't seem worth complicating the test for that, since all we're checking is that an optimisation takes place, and that optimisation isn't related to pointer size. So I think the best fix is just to restrict that part of the test to LP64. Tested on aarch64-linux-gnu, with and without -mabi=ilp32, applied as r276688. Richard 2019-10-08 Richard Sandiford <richard.sandif...@arm.com> gcc/testsuite/ * gcc.target/aarch64/torture/simd-abi-9.c: Require LP64 for the function body test. Index: gcc/testsuite/gcc.target/aarch64/torture/simd-abi-9.c =================================================================== --- gcc/testsuite/gcc.target/aarch64/torture/simd-abi-9.c 2019-09-30 17:46:44.107513831 +0100 +++ gcc/testsuite/gcc.target/aarch64/torture/simd-abi-9.c 2019-10-08 09:23:54.594365710 +0100 @@ -6,7 +6,7 @@ int callee (void); /* -** caller: +** caller: { target lp64 } ** ldr (w[0-9]+), \[x0\] ** cbn?z \1, [^\n]* ** ...