[Bug target/20717] New: [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[Bug target/20717] New: [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[jsm28 at gcc dot gnu dot org]

Many g++ and libstdc++ testcases started failing on mainline on ia64-hpux on
20050331.  The failures are of the form:

ld: (Warning) Unsatisfied symbol "std::basic_stringbuf<char,
std::char_traits<char>, std::allocator<char> >::~basic_stringbuf()" in file
/scratch/gcc/nightly-2005-03-31-mainline/ia64-hp-hpux11.23/build_gcc/build/ia64-hp-hpux11.23/./libstdc++-v3/src/.libs/libstdc++.so

Suspicious patches are

2005-03-31  Jan Hubicka  <jh@suse.cz>

         * decl2.c (finish_objects): Mark ctor as needed.
         (cp_finish_file): Output variables only in nonunit-at-a-time.

(known to be wrong because C++ is always unit-at-a-time)

and

2005-03-31  Jan Hubicka  <jh@suse.cz>

         2004-11-02  Jan Hubicka  <jh@suse.cz>

         * cgraph.c (cgraph_varpool_node_name): New function.
         (dump_cgraph_varpool_node): New function.
         (dump_varpool): New function.
         * cgraphunit.c (cgraph_optimize): Dump varpool.

         ...

This could also be related to <http://gcc.gnu.org/ml/gcc/2005-03/msg01326.html>,
bootstrap failure on ia64-linux.

-- 
           Summary: [4.1 Regression] Many C++ testsuite failures on ia64-
                    hpux
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jsm28 at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org,jh at suse dot cz,mark
                    at codesourcery dot com
GCC target triplet: ia64-*-hpux11.*


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
   Target Milestone|---                         |4.1.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[schwab at suse dot de]

------- Additional Comments From schwab at suse dot de  2005-04-02 19:03 -------
The bootstrap failure on ia64-linux was introduced by the latter change, so it  
is most likely the same issue.  

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |schwab at suse dot de
 GCC target triplet|ia64-*-hpux11.*             |ia64-*-*


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[jh at suse dot cz]

------- Additional Comments From jh at suse dot cz  2005-04-02 19:26 -------
Subject: Re:  [4.1 Regression] Many C++ testsuite failures on ia64-hpux

> 
> ------- Additional Comments From schwab at suse dot de  2005-04-02 19:03 -------
> The bootstrap failure on ia64-linux was introduced by the latter change, so it  
> is most likely the same issue.  

The cgraph change concerning unit-at-a-time and C++ should be noop now,
but it should not affect anything.
I did bootstrapped ia-64 at a time of applying the branch and I saw
failure on clean branch already, so it looks like something broke
earlier.  I will try to investigate after returning from trip at
wenesday.

Honza
> 
> -- 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                  CC|                            |schwab at suse dot de
>  GCC target triplet|ia64-*-hpux11.*             |ia64-*-*
> 
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717
> 
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug, or are watching someone who is.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[wilson at gcc dot gnu dot org]

------- Additional Comments From wilson at gcc dot gnu dot org  2005-04-05 04:04 -------
The patch seems to have been incompletely checked in.  The ChangeLog entry
claims to be removing code from final.c, but the code is still there.

I laboriously tracked down the problem to the fact that cgraphunit handles
ADDR_EXPR but not FDESC_ADDR.  Hence, only targets that use function descriptors
are affected.  Like IA-64.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-04-05 04:04:14
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[jh at suse dot cz]

------- Additional Comments From jh at suse dot cz  2005-04-05 09:46 -------
Subject: Re:  [4.1 Regression] Many C++ testsuite failures on ia64-hpux

> 
> ------- Additional Comments From wilson at gcc dot gnu dot org  2005-04-05 04:04 -------
> The patch seems to have been incompletely checked in.  The ChangeLog entry
> claims to be removing code from final.c, but the code is still there.
> 
> I laboriously tracked down the problem to the fact that cgraphunit handles
> ADDR_EXPR but not FDESC_ADDR.  Hence, only targets that use function descriptors
> are affected.  Like IA-64.

This sounds like reason for the failure.  What precisely is the
behaviour of FDESC_ADDR?  (ie should I behave same way as if I see
ADDR_EXPR - assume that address of the function has been taken and thus
it is neccesary?).

Honza
> 
> 
> -- 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|UNCONFIRMED                 |NEW
>      Ever Confirmed|                            |1
>    Last reconfirmed|0000-00-00 00:00:00         |2005-04-05 04:04:14
>                date|                            |
> 
> 
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717
> 
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug, or are watching someone who is.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[wilson at specifixinc dot com]

------- Additional Comments From wilson at specifixinc dot com  2005-04-05 19:44 -------
Subject: Re:  [4.1 Regression] Many C++ testsuite failures
 on ia64-hpux

jh at suse dot cz wrote:
> This sounds like reason for the failure.  What precisely is the
> behaviour of FDESC_ADDR?  (ie should I behave same way as if I see
> ADDR_EXPR - assume that address of the function has been taken and thus
> it is neccesary?).

On IA-64, the address of a function is a two-word function descriptor, 
one word is the function entry point, and one word is the function's gp 
value.  FDESC_EXPR is used to represent this.  FDESC_EXPR must be used 
in places where the difference matters, e.g. in initializers for 
vtables.  According to the docs in tree.def, operand0 of FDESC_EXPR is 
always a function, so it should be possible to handle FDESC_EXPR exactly 
the same as ADDR_EXPR if you don't care about the difference.  That is 
what the untested patch I added to the PR does.  It seems to work for 
the testcase.  There is also the question of whether there are other 
places that need the same fix.  I haven't looked.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-04-07 04:25 -------
Subject: Bug 20717

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	wilson@gcc.gnu.org	2005-04-07 04:25:09

Modified files:
	gcc            : ChangeLog cgraphunit.c 

Log message:
	Fix numerous IA-64 C++ failures, IA-64 bootstrap trouble.
	PR target/20717
	* cgraphunit.c (record_call_1, case FDESC_EXPR): Handle same as
	ADDR_EXPR.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.8177&r2=2.8178
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cgraphunit.c.diff?cvsroot=gcc&r1=1.99&r2=1.100



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717

[Bug target/20717] [4.1 Regression] Many C++ testsuite failures on ia64-hpux

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-04-07 22:50 -------
Fixed.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20717