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

Reply via email to