public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
@ 2005-02-07  6:03 jsm28 at gcc dot gnu dot org
  2005-02-07 13:06 ` [Bug c++/19797] " pinskia at gcc dot gnu dot org
                   ` (15 more replies)
  0 siblings, 16 replies; 17+ messages in thread
From: jsm28 at gcc dot gnu dot org @ 2005-02-07  6:03 UTC (permalink / raw)
  To: gcc-bugs

The test g++.dg/abi/inline1.C scan-assembler-not _ZTV1S
fails on hppa{2.0w,64}-hp-hpux11.{11,23}.  This is a regression
from 3.4.x.

-- 
           Summary: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-
                    hpux*
           Product: gcc
           Version: 4.0.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c++
        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
GCC target triplet: hppa*-*-hpux11.*


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
@ 2005-02-07 13:06 ` pinskia at gcc dot gnu dot org
  2005-02-13  2:12 ` jsm28 at gcc dot gnu dot org
                   ` (14 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-02-07 13:06 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2005-02-07 01:54 -------
I want to say this is target bug, because there was a change to move hppa*-*-hpux* over to weak 
symbol support.

-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
  2005-02-07 13:06 ` [Bug c++/19797] " pinskia at gcc dot gnu dot org
@ 2005-02-13  2:12 ` jsm28 at gcc dot gnu dot org
  2005-02-13  5:07 ` pinskia at gcc dot gnu dot org
                   ` (13 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: jsm28 at gcc dot gnu dot org @ 2005-02-13  2:12 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From jsm28 at gcc dot gnu dot org  2005-02-12 21:35 -------
Created an attachment (id=8189)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=8189&action=view)
Candidate patch

This patch fixes the problem on hppa64.  However, it causes the 32-bit HP
SOM linker to crash linking libstdc++.	This linker bug has been reported
to HP.	Once a linker patch is available from HP, the patch will be submitted
if it passes testing.


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
  2005-02-07 13:06 ` [Bug c++/19797] " pinskia at gcc dot gnu dot org
  2005-02-13  2:12 ` jsm28 at gcc dot gnu dot org
@ 2005-02-13  5:07 ` pinskia at gcc dot gnu dot org
  2005-02-13 17:25 ` danglin at gcc dot gnu dot org
                   ` (12 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-02-13  5:07 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-02-12 22:11:13
               date|                            |


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (2 preceding siblings ...)
  2005-02-13  5:07 ` pinskia at gcc dot gnu dot org
@ 2005-02-13 17:25 ` danglin at gcc dot gnu dot org
  2005-02-13 17:41 ` danglin at gcc dot gnu dot org
                   ` (11 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu dot org @ 2005-02-13 17:25 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From danglin at gcc dot gnu dot org  2005-02-13 00:59 -------
Weak symbol support was added for SOM in 3.3 on 11.X.  COMDAT was
added in 3.4 but this currently only affects 10.X.  Hppa64 has had
weak support since its inception.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |danglin at gcc dot gnu dot
                   |                            |org


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (3 preceding siblings ...)
  2005-02-13 17:25 ` danglin at gcc dot gnu dot org
@ 2005-02-13 17:41 ` danglin at gcc dot gnu dot org
  2005-02-13 17:58 ` joseph at codesourcery dot com
                   ` (10 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu dot org @ 2005-02-13 17:41 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From danglin at gcc dot gnu dot org  2005-02-13 01:44 -------
1) In the past, we haven't worried about the presence of unnecessary
   .IMPORT directives as they are harmless.  There are several places
   in the testsuite where we work around this problem.

2) I don't believe that putting this fix in the PA backend is the best
   fix.  In December, Zack introduced a mechanism to defer emission of
   assemble externals:

2004-12-08  Zack Weinberg  <zack@codesourcery.com>

        PR 17982
        * varasm.c (pending_assemble_externals): New static.
        (assemble_external_real): Meat of assemble_external split out
        to this new function.
        (process_pending_assemble_externals): New function.
        (assemble_external): Use gcc_assert.  If flag_unit_at_a_time
        is true and the basic test passes, merely cons the decl onto
        the pending list to be handled later.
        * tree.h: Declare process_pending_assemble_externals.
        * cgraphunit.c (cgraph_optimize): Call it.

   Thus, we are adding a hppa fix on top of an ia64 fix to the main code.

   I proposed a backend fix for an AIX problem with respect to assemble
   output defines.  Again, the fix was to defer emission of the output
   defines until the file end.  David Edelsohn I think rightly objected
   that this fix was too invasive and ultimately a fix to the main code
   body was accepted:

2005-01-16  John David Anglin  <dave.anglin@nrc-cnrc.gc.ca>

        PR target/16304
        * defaults.h (TARGET_DEFERRED_OUTPUT_DEFS): Provide default.
        * toplev.c (compile_file): Call process_pending_assemble_output_defs
        just before targetm.asm_out.file_end.
        * tree.h (process_pending_assemble_output_defs): Declare.
        * varasm.c (assemble_output_def, process_pending_assemble_output_defs):
        New functions.
        (assemble_alias): Defer generation of assembly code for defines when
        TARGET_DEFERRED_OUTPUT_DEFS is true.
        * config/rs6000/aix41.h (TARGET_DEFERRED_OUTPUT_DEFS): Define.
        * config/rs6000/aix43.h (TARGET_DEFERRED_OUTPUT_DEFS): Define.
        * doc/tm.texi (TARGET_DEFERRED_OUTPUT_DEFS): document.

   On the mainline, this fix used Nathan's vector support which Mark
   was keen to support.

3) I'm concerned that the proposed fix requires a linker fix.  There won't
   be a fix for HP-UX 10 and possibly 11.00.  There's still interest in these
   system versions for a number of older machines that are still in fairly
   wide use.  


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (4 preceding siblings ...)
  2005-02-13 17:41 ` danglin at gcc dot gnu dot org
@ 2005-02-13 17:58 ` joseph at codesourcery dot com
  2005-02-13 17:58 ` danglin at gcc dot gnu dot org
                   ` (9 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: joseph at codesourcery dot com @ 2005-02-13 17:58 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From joseph at codesourcery dot com  2005-02-13 02:10 -------
Subject: Re:  [4.0 Regression] g++.dg/abi/inline1.C fails on
 hppa*-*-hpux*

On Sun, 13 Feb 2005, danglin at gcc dot gnu dot org wrote:

> 1) In the past, we haven't worried about the presence of unnecessary
>    .IMPORT directives as they are harmless.  There are several places
>    in the testsuite where we work around this problem.

This bug is, however, a regression, and as such should be fixed for 4.0.

> 2) I don't believe that putting this fix in the PA backend is the best
>    fix.  In December, Zack introduced a mechanism to defer emission of
>    assemble externals:
> 
> 2004-12-08  Zack Weinberg  <zack@codesourcery.com>
> 
>         PR 17982
>         * varasm.c (pending_assemble_externals): New static.
>         (assemble_external_real): Meat of assemble_external split out
>         to this new function.
>         (process_pending_assemble_externals): New function.
>         (assemble_external): Use gcc_assert.  If flag_unit_at_a_time
>         is true and the basic test passes, merely cons the decl onto
>         the pending list to be handled later.
>         * tree.h: Declare process_pending_assemble_externals.
>         * cgraphunit.c (cgraph_optimize): Call it.
> 
>    Thus, we are adding a hppa fix on top of an ia64 fix to the main code.

Zack's view was that changing the core code to defer this until the end of 
assembler output for all platforms would be appropriate for 4.1 - but 
fixing in the PA backend, as is done in the IA64 backend, though not an 
ideal place and involving double deferral by both core code and backend, 
makes more sense in the context of fixing a regression for 4.0.

> 3) I'm concerned that the proposed fix requires a linker fix.  There won't
>    be a fix for HP-UX 10 and possibly 11.00.  There's still interest in these
>    system versions for a number of older machines that are still in fairly
>    wide use.  

We don't know whether a linker fix is required for those versions (or 
indeed for 11.11, as when testing I tested the patch for 64-bit 11.11 and 
32-bit 11.23 simultaneously on different systems).  The linker problem 
could just happen to be triggered some combination of circumstances that 
happen to arise for libstdc++ on a particular HP-UX release after this 
patch; or when the linker bug is fixed it could turn out that actually 
there is something wrong with the object files generated (which however 
should never cause the linker to crash), whether because of a bug in this 
patch or a bug in the GNU assembler, either of which can be fixed.  I 
don't know whether GCC mainline does in fact work on those old HP-UX 
versions - I can only test on PA 11.11 and 11.23 and IA64 11.22 and 11.23 
- but there must always be a risk if the GNU tools do not support a system 
well enough and the system tools are no longer maintained that new GCC 
versions will run into bugs with the system tools.

If necessary, with the deferral in the back end it is possible to arrange 
for 11.00 and 10.x to use the old behavior while more recent versions 
expect a linker fix - something not so readily possible with deferral 
until end of output happening unconditionally for all targets in the core 
code.



-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (5 preceding siblings ...)
  2005-02-13 17:58 ` joseph at codesourcery dot com
@ 2005-02-13 17:58 ` danglin at gcc dot gnu dot org
  2005-02-13 18:00 ` joseph at codesourcery dot com
                   ` (8 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu dot org @ 2005-02-13 17:58 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From danglin at gcc dot gnu dot org  2005-02-13 02:12 -------
Looking at the testsuite results, this regression appears to have been
introduced between Jul 28 22:04:59 UTC and Jul 29 23:04:16 UTC.


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (6 preceding siblings ...)
  2005-02-13 17:58 ` danglin at gcc dot gnu dot org
@ 2005-02-13 18:00 ` joseph at codesourcery dot com
  2005-02-13 18:04 ` dave at hiauly1 dot hia dot nrc dot ca
                   ` (7 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: joseph at codesourcery dot com @ 2005-02-13 18:00 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From joseph at codesourcery dot com  2005-02-13 02:24 -------
Subject: Re:  [4.0 Regression] g++.dg/abi/inline1.C fails on
 hppa*-*-hpux*

On Sun, 13 Feb 2005, danglin at gcc dot gnu dot org wrote:

> Looking at the testsuite results, this regression appears to have been
> introduced between Jul 28 22:04:59 UTC and Jul 29 23:04:16 UTC.

That's when the test was added - but though the test is new since 3.4, 
it's still a regression as it passes when built with 3.4.3.  
Unsurprisingly it probably wasn't tested on a platform defining 
ASM_OUTPUT_EXTERNAL, since most platforms don't.  It is quite possible 
that that large patch "C++ PATCH: Cleanup linkage handling" did cause the 
problem with the test as well - it would be necessary to test compiling 
the testcase with compilers from before and after the change to find out, 
given that my other endpoint is that 3.4.3 compiles the test to assembler 
without the unwanted .IMPORT - but if so then it's likely a side effect of 
improving correctness in the C++ front end.  (Though of course if there is 
a simple fix within the C++ front end which allows the question of 
deferring ASM_OUTPUT_EXTERNAL to be put off to 4.1, that would be 
convenient.)



-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (7 preceding siblings ...)
  2005-02-13 18:00 ` joseph at codesourcery dot com
@ 2005-02-13 18:04 ` dave at hiauly1 dot hia dot nrc dot ca
  2005-02-15 15:55 ` dave at hiauly1 dot hia dot nrc dot ca
                   ` (6 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: dave at hiauly1 dot hia dot nrc dot ca @ 2005-02-13 18:04 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From dave at hiauly1 dot hia dot nrc dot ca  2005-02-13 02:57 -------
Subject: Re:  [4.0 Regression] g++.dg/abi/inline1.C fails on hp

> On Sun, 13 Feb 2005, danglin at gcc dot gnu dot org wrote:
> 
> > Looking at the testsuite results, this regression appears to have been
> > introduced between Jul 28 22:04:59 UTC and Jul 29 23:04:16 UTC.
> 
> That's when the test was added - but though the test is new since 3.4, 
> it's still a regression as it passes when built with 3.4.3.  
> Unsurprisingly it probably wasn't tested on a platform defining 
> ASM_OUTPUT_EXTERNAL, since most platforms don't.  It is quite possible 
> that that large patch "C++ PATCH: Cleanup linkage handling" did cause the 
> problem with the test as well - it would be necessary to test compiling 
> the testcase with compilers from before and after the change to find out, 
> given that my other endpoint is that 3.4.3 compiles the test to assembler 
> without the unwanted .IMPORT - but if so then it's likely a side effect of 
> improving correctness in the C++ front end.  (Though of course if there is 
> a simple fix within the C++ front end which allows the question of 
> deferring ASM_OUTPUT_EXTERNAL to be put off to 4.1, that would be 
> convenient.)

The above cleanup broke linkage support on systems without weak or
one-only support.  There is an AIX PR on this.

I have access to an old version of the HP linker code which was placed
into the public domain.  This has been useful in a number of cases in
diagnosing problems.

I do test HP-UX 10.20 and 11.00, but not on a daily basis.  As far as
I know, there aren't any significant problems with 4.0.0 beyond that
present in 11.11.

I currently have PHSS_30968 on the HP-UX 11.11 system that I test
on.  A linker patch is needed on 11.X to successfully build libstdc++
but I assume you are aware of that.  I'll give your change a try
when my current build completes.

Dave


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (8 preceding siblings ...)
  2005-02-13 18:04 ` dave at hiauly1 dot hia dot nrc dot ca
@ 2005-02-15 15:55 ` dave at hiauly1 dot hia dot nrc dot ca
  2005-02-21  0:35 ` pinskia at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: dave at hiauly1 dot hia dot nrc dot ca @ 2005-02-15 15:55 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From dave at hiauly1 dot hia dot nrc dot ca  2005-02-15 01:56 -------
Subject: Re:  [4.0 Regression] g++.dg/abi/inline1.C fails on hp

> > 3) I'm concerned that the proposed fix requires a linker fix.  There won't
> >    be a fix for HP-UX 10 and possibly 11.00.  There's still interest in
> these
> >    system versions for a number of older machines that are still in fairly
> >    wide use.  
> 
> We don't know whether a linker fix is required for those versions (or 
> indeed for 11.11, as when testing I tested the patch for 64-bit 11.11 and 
> 32-bit 11.23 simultaneously on different systems).  The linker problem 
> could just happen to be triggered some combination of circumstances that 
> happen to arise for libstdc++ on a particular HP-UX release after this 
> patch; or when the linker bug is fixed it could turn out that actually 
> there is something wrong with the object files generated (which however 
> should never cause the linker to crash), whether because of a bug in this 
> patch or a bug in the GNU assembler, either of which can be fixed.

The linker doesn't fault if this line is removed:

      if (!TREE_ASM_WRITTEN (decl) && TREE_SYMBOL_REFERENCED (id))

I believe that this indicates that a necessary .import statement
has been removed.

Dave


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (9 preceding siblings ...)
  2005-02-15 15:55 ` dave at hiauly1 dot hia dot nrc dot ca
@ 2005-02-21  0:35 ` pinskia at gcc dot gnu dot org
  2005-02-25  8:27 ` jsm28 at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-02-21  0:35 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2005-02-20 17:52 -------
hppa-hpux is a primary/secondary target.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.0.0


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (10 preceding siblings ...)
  2005-02-21  0:35 ` pinskia at gcc dot gnu dot org
@ 2005-02-25  8:27 ` jsm28 at gcc dot gnu dot org
  2005-02-25 15:35 ` dave at hiauly1 dot hia dot nrc dot ca
                   ` (3 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: jsm28 at gcc dot gnu dot org @ 2005-02-25  8:27 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From jsm28 at gcc dot gnu dot org  2005-02-24 23:49 -------
Even with the patch

2005-02-20  John David Anglin  <dave.anglin@nrc-cnrc.gc.ca>

        * pa.c (pa_assemble_integer, get_plabel): Small cleanups.
        (pa_asm_output_mi_thunk): Use asm_output_insn instead of fprintf.
        * som.h (ASM_OUTPUT_EXTERNAL_LIBCALL): Small cleanup.

present in the tree, I still get a linker crash on hppa2.0w-hp-hpux11.11
with my patch as attached to this PR.


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (11 preceding siblings ...)
  2005-02-25  8:27 ` jsm28 at gcc dot gnu dot org
@ 2005-02-25 15:35 ` dave at hiauly1 dot hia dot nrc dot ca
  2005-03-05  1:41 ` [Bug c++/19797] [4.0/4.1 " cvs-commit at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: dave at hiauly1 dot hia dot nrc dot ca @ 2005-02-25 15:35 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From dave at hiauly1 dot hia dot nrc dot ca  2005-02-25 04:37 -------
Subject: Re:  [4.0 Regression] g++.dg/abi/inline1.C fails on hp

> It builds successfully on hppa64-hp-hpux11.11.  There seem to be some
> regression in the gfortran suite but none elsewhere.  I don't know if
> the fortran regressions are related or not to the change.

These regressions were likely caused by this problem
<http://gcc.gnu.org/ml/fortran/2005-02/msg00318.html>.

I've found another problem with the original patch.  Root GTY objects
can't be inside an #ifdef.  This causes a build failure on linux.

Dave


-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0/4.1 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (12 preceding siblings ...)
  2005-02-25 15:35 ` dave at hiauly1 dot hia dot nrc dot ca
@ 2005-03-05  1:41 ` cvs-commit at gcc dot gnu dot org
  2005-03-05  1:59 ` cvs-commit at gcc dot gnu dot org
  2005-03-05  3:14 ` danglin at gcc dot gnu dot org
  15 siblings, 0 replies; 17+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2005-03-05  1:41 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-03-05 01:41 -------
Subject: Bug 19797

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-4_0-branch
Changes by:	danglin@gcc.gnu.org	2005-03-05 01:40:48

Modified files:
	gcc            : ChangeLog 
	gcc/testsuite  : ChangeLog 
	gcc/config/pa  : elf.h pa-protos.h pa.c pa.h pa64-hpux.h som.h 
	gcc/testsuite/g++.dg/abi: vague1.C 

Log message:
	PR c++/19797
	* elf.h, pa64-hpux.h, som.h (ASM_OUTPUT_EXTERNAL_REAL): Define.
	(ASM_OUTPUT_EXTERNAL): Call pa_hpux_asm_output_external.
	* pa-protos.h (pa_hpux_asm_output_external): Add prototype.
	* pa.c (get_plabel): Change argument to symbol_ref.  Call
	maybe_get_identifier instead of get_identifier.
	(pa_hpux_asm_output_external, pa_hpux_file_end): New functions.
	(struct deferred_plabel): Replace name field with symbol field.
	(TARGET_ASM_FILE_END): Define as pa_hpux_file_end if
	ASM_OUTPUT_EXTERNAL_REAL is defined.
	(output_global_address): Use output_addr_const for output of all
	symbol_refs.
	(output_deferred_plabels): Use symbol_ref instead of name for address
	output.
	* pa.h (SYMBOL_FLAG_REFERENCED, SYMBOL_REF_REFERENCED_P,
	ASM_OUTPUT_SYMBOL_REF): New macros.
	* som.h (ASM_OUTPUT_EXTERNAL_LIBCALL): Use assemble_name_raw.  Update
	comment.
	(ASM_WEAKEN_LABEL): Use targetm.asm_out.globalize_label to globalize
	label.
	* g++.dg/abi/vague1.C: Remove xfail hppa*-*-hpux*.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=2.7592.2.23&r2=2.7592.2.24
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.5084.2.16&r2=1.5084.2.17
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/elf.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.15.72.1&r2=1.15.72.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa-protos.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.40&r2=1.40.8.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa.c.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.287&r2=1.287.2.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.242.8.1&r2=1.242.8.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa64-hpux.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.39.8.1&r2=1.39.8.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/som.h.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.55.2.1&r2=1.55.2.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/abi/vague1.C.diff?cvsroot=gcc&only_with_tag=gcc-4_0-branch&r1=1.3&r2=1.3.76.1



-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0/4.1 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (13 preceding siblings ...)
  2005-03-05  1:41 ` [Bug c++/19797] [4.0/4.1 " cvs-commit at gcc dot gnu dot org
@ 2005-03-05  1:59 ` cvs-commit at gcc dot gnu dot org
  2005-03-05  3:14 ` danglin at gcc dot gnu dot org
  15 siblings, 0 replies; 17+ messages in thread
From: cvs-commit at gcc dot gnu dot org @ 2005-03-05  1:59 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-03-05 01:59 -------
Subject: Bug 19797

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	danglin@gcc.gnu.org	2005-03-05 01:59:24

Modified files:
	gcc            : ChangeLog 
	gcc/testsuite  : ChangeLog 
	gcc/config/pa  : elf.h pa-protos.h pa.c pa.h pa64-hpux.h som.h 
	gcc/testsuite/g++.dg/abi: vague1.C 

Log message:
	PR c++/19797
	* elf.h, pa64-hpux.h, som.h (ASM_OUTPUT_EXTERNAL_REAL): Define.
	(ASM_OUTPUT_EXTERNAL): Call pa_hpux_asm_output_external.
	* pa-protos.h (pa_hpux_asm_output_external): Add prototype.
	* pa.c (get_plabel): Change argument to symbol_ref.  Call
	maybe_get_identifier instead of get_identifier.
	(pa_hpux_asm_output_external, pa_hpux_file_end): New functions.
	(struct deferred_plabel): Replace name field with symbol field.
	(TARGET_ASM_FILE_END): Define as pa_hpux_file_end if
	ASM_OUTPUT_EXTERNAL_REAL is defined.
	(output_global_address): Use output_addr_const for output of all
	symbol_refs.
	(output_deferred_plabels): Use symbol_ref instead of name for address
	output.
	* pa.h (SYMBOL_FLAG_REFERENCED, SYMBOL_REF_REFERENCED_P,
	ASM_OUTPUT_SYMBOL_REF): New macros.
	* som.h (ASM_OUTPUT_EXTERNAL_LIBCALL): Use assemble_name_raw.  Update
	comment.
	(ASM_WEAKEN_LABEL): Use targetm.asm_out.globalize_label to globalize
	label.
	* g++.dg/abi/vague1.C: Remove xfail hppa*-*-hpux*.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.7668&r2=2.7669
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.5110&r2=1.5111
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/elf.h.diff?cvsroot=gcc&r1=1.16&r2=1.17
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa-protos.h.diff?cvsroot=gcc&r1=1.40&r2=1.41
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa.c.diff?cvsroot=gcc&r1=1.287&r2=1.288
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa.h.diff?cvsroot=gcc&r1=1.243&r2=1.244
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/pa64-hpux.h.diff?cvsroot=gcc&r1=1.40&r2=1.41
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/pa/som.h.diff?cvsroot=gcc&r1=1.56&r2=1.57
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/abi/vague1.C.diff?cvsroot=gcc&r1=1.3&r2=1.4



-- 


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug c++/19797] [4.0/4.1 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux*
  2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
                   ` (14 preceding siblings ...)
  2005-03-05  1:59 ` cvs-commit at gcc dot gnu dot org
@ 2005-03-05  3:14 ` danglin at gcc dot gnu dot org
  15 siblings, 0 replies; 17+ messages in thread
From: danglin at gcc dot gnu dot org @ 2005-03-05  3:14 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From danglin at gcc dot gnu dot org  2005-03-05 03:14 -------
Fixed in CVS <http://gcc.gnu.org/ml/gcc-patches/2005-03/msg00440.html>.


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


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


^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2005-03-05  3:14 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-02-07  6:03 [Bug c++/19797] New: [4.0 Regression] g++.dg/abi/inline1.C fails on hppa*-*-hpux* jsm28 at gcc dot gnu dot org
2005-02-07 13:06 ` [Bug c++/19797] " pinskia at gcc dot gnu dot org
2005-02-13  2:12 ` jsm28 at gcc dot gnu dot org
2005-02-13  5:07 ` pinskia at gcc dot gnu dot org
2005-02-13 17:25 ` danglin at gcc dot gnu dot org
2005-02-13 17:41 ` danglin at gcc dot gnu dot org
2005-02-13 17:58 ` joseph at codesourcery dot com
2005-02-13 17:58 ` danglin at gcc dot gnu dot org
2005-02-13 18:00 ` joseph at codesourcery dot com
2005-02-13 18:04 ` dave at hiauly1 dot hia dot nrc dot ca
2005-02-15 15:55 ` dave at hiauly1 dot hia dot nrc dot ca
2005-02-21  0:35 ` pinskia at gcc dot gnu dot org
2005-02-25  8:27 ` jsm28 at gcc dot gnu dot org
2005-02-25 15:35 ` dave at hiauly1 dot hia dot nrc dot ca
2005-03-05  1:41 ` [Bug c++/19797] [4.0/4.1 " cvs-commit at gcc dot gnu dot org
2005-03-05  1:59 ` cvs-commit at gcc dot gnu dot org
2005-03-05  3:14 ` danglin at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).