egcs-971127: results on powerpc-ibm-aix4.2.1.0

egcs-971127: results on powerpc-ibm-aix4.2.1.0

[Andrey Slepuhin]

Here are the results of egcs-971127 on powerpc-ibm-aix4.2.1.0.
egcs native configuration with --enable-shared.
Used patches: patch for multithreaded exception handling
by Teemu Torma and my patch for shared libraries support.
Why there is a regression in gcc testsuite
(gcc.c-torture/execute/va-arg-5.c)?
----------------------------------------------------------
Test Run By pooh on Sat Nov 29 14:15:04 1997
Native configuration is powerpc-ibm-aix4.2.1.0

		=== gcc tests ===

FAIL: gcc.c-torture/compile/920501-4.c,  -O2  
FAIL: gcc.c-torture/compile/920501-4.c,  -O2 -fomit-frame-pointer
-finline-functions  
FAIL: gcc.c-torture/compile/920501-4.c,  -O2 -fomit-frame-pointer
-finline-functions -funroll-loops  
FAIL: gcc.c-torture/compile/920501-4.c,  -O2 -fomit-frame-pointer
-finline-functions -funroll-all-loops  
FAIL: gcc.c-torture/execute/931004-11.c execution,  -O0 
FAIL: gcc.c-torture/execute/931004-11.c execution,  -O1 
FAIL: gcc.c-torture/execute/931004-11.c execution,  -O2 
FAIL: gcc.c-torture/execute/931004-12.c execution,  -O0 
FAIL: gcc.c-torture/execute/931004-12.c execution,  -O1 
FAIL: gcc.c-torture/execute/931004-12.c execution,  -O2 
FAIL: gcc.c-torture/execute/931004-12.c execution,  -O2 -fomit-frame-pointer
-finline-functions 
FAIL: gcc.c-torture/execute/931004-12.c execution,  -O2 -fomit-frame-pointer
-finline-functions -funroll-loops 
FAIL: gcc.c-torture/execute/931004-12.c execution,  -O2 -fomit-frame-pointer
-finline-functions -funroll-all-loops 
FAIL: gcc.c-torture/execute/complex-5.c execution,  -O2 -fomit-frame-pointer
-finline-functions 
FAIL: gcc.c-torture/execute/va-arg-5.c execution,  -O0 
FAIL: gcc.c-torture/execute/va-arg-5.c execution,  -O1 
FAIL: gcc.c-torture/execute/va-arg-5.c execution,  -O2 
FAIL: gcc.c-torture/execute/va-arg-5.c execution,  -O2 -fomit-frame-pointer
-finline-functions 

		=== gcc Summary ===

# of expected passes		4865
# of unexpected failures	18
# of expected failures		5
# of unsupported tests		7

		=== g++ tests ===

XPASS: g++.jason/destruct3.C - (test for bogus messages, line 38)
FAIL: g++.jason/thunk3.C (test for excess errors)
FAIL: g++.law/profile1.C (test for excess errors)
XPASS: g++.mike/dyncast1.C  Execution test
XPASS: g++.mike/dyncast2.C  Execution test
FAIL: g++.pt/typename5.C (test for excess errors)

		=== g++ Summary ===

# of expected passes		3395
# of unexpected failures	3
# of unexpected successes	3
# of expected failures		82
# of untested testcases		6

		=== g77 tests ===

		=== g77 Summary ===

# of expected passes		132

		=== libio tests ===

		=== libio Summary ===

# of expected passes		40

		=== libstdc++ tests ===

		=== libstdc++ Summary ===

# of expected passes		30

Re: egcs-971127: results on powerpc-ibm-aix4.2.1.0

[Jeffrey A Law]

  In message < 34801C0C.A67C5FAB@msu.ru >you write:
  > Why there is a regression in gcc testsuite
  > (gcc.c-torture/execute/va-arg-5.c)?
Bug in Meissner's rework of some of Geoffrey Keating's patches;

This should fix the problem (and will be included in the release).

Index: rs6000.c
===================================================================
RCS file: /cvs/cvsfiles/egcs/gcc/config/rs6000/rs6000.c,v
retrieving revision 1.3.2.4
retrieving revision 1.3.2.5
diff -c -3 -p -r1.3.2.4 -r1.3.2.5
*** rs6000.c	1997/11/27 00:06:51	1.3.2.4
--- rs6000.c	1997/11/29 21:37:41	1.3.2.5
*************** function_arg_partial_nregs (cum, mode, t
*** 1387,1395 ****
  	return 0;
      }
  
-   if (GET_MODE_CLASS (mode) == MODE_FLOAT && TARGET_HARD_FLOAT)
-     return 0;
- 
    if (cum->words < GP_ARG_NUM_REG
        && GP_ARG_NUM_REG < (cum->words + RS6000_ARG_SIZE (mode, type, named)))
      {
--- 1387,1392 ----

Re: egcs-971127: results on powerpc-ibm-aix4.2.1.0

[Andrey Slepuhin]

Jeffrey A Law wrote:
>   In message < 34801C0C.A67C5FAB@msu.ru >you write:
>   > Why there is a regression in gcc testsuite
>   > (gcc.c-torture/execute/va-arg-5.c)?
> Bug in Meissner's rework of some of Geoffrey Keating's patches;
> 
> This should fix the problem (and will be included in the release).

Checked. Ok.

Andrey.

Re: egcs-971127: results on powerpc-ibm-aix4.2.1.0

[Franz Sirl]

At 22:26 Uhr -0000 29.11.1997, Jeffrey A Law wrote:
>  In message < 34801C0C.A67C5FAB@msu.ru >you write:
>  > Why there is a regression in gcc testsuite
>  > (gcc.c-torture/execute/va-arg-5.c)?
>Bug in Meissner's rework of some of Geoffrey Keating's patches;
>
>This should fix the problem (and will be included in the release).
>
>Index: rs6000.c
>===================================================================
>RCS file: /cvs/cvsfiles/egcs/gcc/config/rs6000/rs6000.c,v
>retrieving revision 1.3.2.4
>retrieving revision 1.3.2.5
>diff -c -3 -p -r1.3.2.4 -r1.3.2.5
>*** rs6000.c	1997/11/27 00:06:51	1.3.2.4
>--- rs6000.c	1997/11/29 21:37:41	1.3.2.5
>*************** function_arg_partial_nregs (cum, mode, t
>*** 1387,1395 ****
>  	return 0;
>      }
>
>-   if (GET_MODE_CLASS (mode) == MODE_FLOAT && TARGET_HARD_FLOAT)
>-     return 0;
>-
>    if (cum->words < GP_ARG_NUM_REG
>        && GP_ARG_NUM_REG < (cum->words + RS6000_ARG_SIZE (mode, type,
>named)))
>      {
>--- 1387,1392 ----

Hmm, this also solves a sig11 while compiling glibc-2.1 for me, but it
didn't change the test behaviour on powerpc-unknown-linux-gnulibc1. The
strange thing is, after getting the sig11 I started applying Geoff's
remaining patches step by step and the following one also prohibited the
sig11. Is Michaels rework really buggy?

BTW, while looking at *.s files generated by -save-temps I noticed that I
can't switch on -mmultiple, it is always directly followed by
-mno-multiple. I tried with various combinations of -mpowerpc,
-mcpu=powerpc, etc. and was never able to get rid of the -mno-multiple.
According to the docs this should be possible. Is there any other option
that forces -mno-multiple?

Anyway, egcs-971127 is now quite fit for release from the Linux/PPC
viewpoint ;-). Thanks all the helpful guys involved in this!

Bye,
Franz.

Patch by Geoff Keating, originally for an older version of egcs, so it
applies with offset:

* PIC code breaks when nested functions are used; for instance, the following
  compiled with 'cc1 -O2 -fpic -Wall'.  (The -Wall gives a nice visible crash,
  rather than quietly generating invalid code.)

extern void x1(int), x2(int);
extern int c1, c2;
void x3 (int p) {
  static inline void i1 (void) { x1(c1); }
  static inline void i2 (void) { i1(); }
  if (p) i2();
  x2(c2);
}

--- config/rs6000/rs6000.c~     Thu Aug 28 00:39:28 1997
+++ config/rs6000/rs6000.c      Fri Aug 29 01:16:00 1997
@@ -2292,6 +2292,12 @@

   free (machine);
   p->machine = (struct machine_function *)0;
+
+  /* If the nested function used the GOT, pic_offset_table_rtx would
+     have been set to the hard register that the function was using as
+     a pointer into the GOT table.  Reset it to NULL so that if this
+     function uses the GOT it will get GOT_TOC_REGNUM.  */
+  pic_offset_table_rtx = NULL;
 }

 /* Do anything needed before RTL is emitted for each function.  */

Re: egcs-971127: results on powerpc-ibm-aix4.2.1.0

[Jeffrey A Law]

  In message < v03110700b0a7a03e9ac0@[192.149.90.247] >you write:
  > Hmm, this also solves a sig11 while compiling glibc-2.1 for me
Good.

  > but it didn't change the test behaviour on powerpc-unknown-linux-gnulibc1.
It shouldn't.  The change I forwarded to you should not effect ppc-linux
at all.


  > The
  > strange thing is, after getting the sig11 I started applying Geoff's
  > remaining patches step by step and the following one also prohibited the
  > sig11. Is Michaels rework really buggy?
Yes.  It had a real bug.

  > BTW, while looking at *.s files generated by -save-temps I noticed that I
  > can't switch on -mmultiple, it is always directly followed by
  > -mno-multiple. I tried with various combinations of -mpowerpc,
  > -mcpu=powerpc, etc. and was never able to get rid of the -mno-multiple.
  > According to the docs this should be possible. Is there any other option
  > that forces -mno-multiple?
The way the ppc handles this is a little odd -- basically I don't think you
can depend on the info in the .s file to correctly tell you the right thing
for -mmultiple.

  > * PIC code breaks when nested functions are used; for instance, the
  > following compiled with 'cc1 -O2 -fpic -Wall'.  (The -Wall gives a nice
  > visible crash, rather than quietly generating invalid code.)
I rewrote this fix to be more correct.  You shouldn't need Geoff's fix
for this problem anymore.  My version has been in egcs snapshots for
several weeks.

jeff

haifa-enabled egcs-971127: results on powerpc-ibm-aix4.2.1.0

[Andrey Slepuhin]

Here are the results of egcs-971127 on powerpc-ibm-aix4.2.1.0.
egcs native configuration with --enable-shared --enable-haifa.
Used patches: patch for multithreaded exception handling
by Teemu Torma and my patch for shared libraries support.
all tests results are the same as without haifa except
g++ results.
----------------------------------------------------------
Test Run By pooh on Mon Dec  1 17:02:15 1997
Native configuration is powerpc-ibm-aix4.2.1.0

                === g++ tests ===

Running target unix
Using /local/egcs/bin/../share/dejagnu/baseboards/unix.exp as board
description file for target.
Using /local/egcs/bin/../share/dejagnu/config/unix.exp as generic
interface file for target.
Using /local/src/egcs-971127/gcc/testsuite/config/default.exp as
tool-and-target-specific interface file.
Running /local/src/egcs-971127/gcc/testsuite/g++.old-deja/old-deja.exp
...
FAIL: g++.brendan/crash19.C candidate for bad call (test for errors,
line 1418)
FAIL: g++.brendan/crash19.C list of candidates for bad call (test for
errors, line 1463)
FAIL: g++.brendan/crash19.C no matching fn for call (test for errors,
line 1486)
FAIL: g++.brendan/crash19.C invalid type for default argument -- not
expecting const (test for errors, line 1491)
FAIL: g++.brendan/crash48.C candidates (test for errors, line 14)
FAIL: g++.brendan/crash48.C no match (test for errors, line 20)
XPASS: g++.jason/destruct3.C - (test for bogus messages, line 38)
FAIL: g++.jason/thunk3.C (test for excess errors)
FAIL: g++.law/profile1.C (test for excess errors)
XPASS: g++.mike/dyncast1.C  Execution test
XPASS: g++.mike/dyncast2.C  Execution test
FAIL: g++.mike/eh49.C  Execution test
FAIL: g++.pt/typename5.C (test for excess errors)

                === g++ Summary ===

# of expected passes            3388
# of unexpected failures        10
# of unexpected successes       3
# of expected failures          82
# of untested testcases         6