On Thu, Jan 18, 2018 at 08:10:50AM -0500, John David Anglin wrote: > On 2018-01-17 3:07 PM, Jakub Jelinek wrote: > > PR83882 complains that PR81715 testcase fails on callee copies parameter > > targets. The following patch ought to fix that, but I have only > > bootstrapped/regtested it on x86_64-linux and i686-linux + on the testcase > > with hppa. > > > > John, do you think you could test this on hppa without the callee copies > > default change? > On hppa2.0w-hp-hpux11.11, pr81715.C now passes. > > However, there are some new fails in the gcc and g++ suites:
That is really weird, at least in the cross-compiler on throw-1.C I don't see with -mno-caller-copies the new code triggering at all. Tried both hppa-linux and hppa-hpux crosses. Furthermore, throw-1.C has: /* { dg-skip-if "Stack alignment is too small" { hppa*-*-hpux* } } */ so I wonder how it comes you see them run on hpux at all. I don't have hpux or hppa-linux fenv.h, so can't check pr52451.c. What kind of error would be on pr83487-*? If any of those are really caused by my patch, I'd be interest to see e.g. *.gimple and *.optimized dumps with and without the patch (rather than reverting the whole patch, it should be enough to hand edit in function.c the && flag_stack_reuse != SR_NONE part and change it to && 0 && flag_stack_reuse != SR_NONE or similar. > FAIL: g++.dg/torture/stackalign/throw-1.C -O0 execution test > FAIL: g++.dg/torture/stackalign/throw-1.C -O0 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-1.C -O1 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-1.C -O2 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-1.C -O3 -fomit-frame-pointer > -funroll-loops -fpeel-loops -ftracer -finline-functions -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-1.C -O3 -g -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-1.C -Os -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -O0 execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -O0 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -O1 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -O2 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -O3 -fomit-frame-pointer > -funroll-loops -fpeel-loops -ftracer -finline-functions -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -O3 -g -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-2.C -Os -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -O0 execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -O0 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -O1 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -O2 -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -O3 -fomit-frame-pointer > -funroll-loops -fpeel-loops -ftracer -finline-functions -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -O3 -g -fpic execution test > FAIL: g++.dg/torture/stackalign/throw-4.C -Os -fpic execution test > > FAIL: gcc.dg/torture/pr52451.c -O0 execution test > FAIL: gcc.dg/torture/pr52451.c -O1 execution test > FAIL: gcc.dg/torture/pr52451.c -O2 execution test > FAIL: gcc.dg/torture/pr52451.c -O3 -g execution test > FAIL: gcc.dg/torture/pr52451.c -Os execution test > > FAIL: gcc.dg/compat/pr83487-1 c_compat_y_tst.o compile > FAIL: gcc.dg/compat/pr83487-2 c_compat_y_tst.o compile > > Dave > > -- > John David Anglin dave.ang...@bell.net Jakub