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 [email protected]
Jakub
