public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
@ 2011-11-11 13:13 dominiq at lps dot ens.fr
  2011-11-11 13:19 ` [Bug bootstrap/51094] " ro at gcc dot gnu.org
                   ` (29 more replies)
  0 siblings, 30 replies; 31+ messages in thread
From: dominiq at lps dot ens.fr @ 2011-11-11 13:13 UTC (permalink / raw)
  To: gcc-bugs

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

             Bug #: 51094
           Summary: [4.7 Regression] Bootstrap failure at revision 181279
                    on non-ELF targets
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: dominiq@lps.ens.fr
                CC: jason@gcc.gnu.org


Revision 181279 breaks bootstrap on non-ELF targets (e.g., *-*-darwin*):

gcc -c   -g -fkeep-inline-functions -DIN_GCC   -W -Wall -Wno-narrowing
-Wwrite-strings -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
-Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros
-Wno-overlength-strings -Wold-style-definition -Wc++-compat -fno-common 
-DHAVE_CONFIG_H -I. -I. -I../../work/gcc -I../../work/gcc/.
-I../../work/gcc/../include -I./../intl -I../../work/gcc/../libcpp/include
-I/opt/mp/include  -I../../work/gcc/../libdecnumber
-I../../work/gcc/../libdecnumber/dpd -I../libdecnumber  -I/opt/mp/include
-DCLOOG_INT_GMP -DCLOOG_ORG  ../../work/gcc/varasm.c -o varasm.o
../../work/gcc/varasm.c: In function 'default_elf_asm_output_limited_string':
../../work/gcc/varasm.c:7534:16: error: 'ELF_ASCII_ESCAPES' undeclared (first
use in this function)
../../work/gcc/varasm.c:7534:16: note: each undeclared identifier is reported
only once for each function it appears in
../../work/gcc/varasm.c: In function 'default_elf_asm_output_ascii':
../../work/gcc/varasm.c:7586:42: error: 'ELF_STRING_LIMIT' undeclared (first
use in this function)
../../work/gcc/varasm.c:7601:20: error: 'ASCII_DATA_ASM_OP' undeclared (first
use in this function)
../../work/gcc/varasm.c:7601:38: error: expected ')' before string constant
../../work/gcc/varasm.c:7601:38: error: too few arguments to function 'fputs'
../../work/gcc/varasm.c:7604:20: error: 'ELF_ASCII_ESCAPES' undeclared (first
use in this function)

(see also http://gcc.gnu.org/ml/gcc/2011-11/msg00273.html ).


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
@ 2011-11-11 13:19 ` ro at gcc dot gnu.org
  2011-11-11 15:39 ` jason at gcc dot gnu.org
                   ` (28 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: ro at gcc dot gnu.org @ 2011-11-11 13:19 UTC (permalink / raw)
  To: gcc-bugs

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

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jimis at gmx dot net, ro at
                   |                            |gcc dot gnu.org

--- Comment #1 from Rainer Orth <ro at gcc dot gnu.org> 2011-11-11 13:13:11 UTC ---
Dimitrios, could you pleases have a look?


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
  2011-11-11 13:19 ` [Bug bootstrap/51094] " ro at gcc dot gnu.org
@ 2011-11-11 15:39 ` jason at gcc dot gnu.org
  2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
                   ` (27 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jason at gcc dot gnu.org @ 2011-11-11 15:39 UTC (permalink / raw)
  To: gcc-bugs

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2011-11-11
     Ever Confirmed|0                           |1

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-11 15:30:50 UTC ---
Is it fixed now?


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
  2011-11-11 13:19 ` [Bug bootstrap/51094] " ro at gcc dot gnu.org
  2011-11-11 15:39 ` jason at gcc dot gnu.org
@ 2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
                   ` (26 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2011-11-11 15:41 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #4 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> 2011-11-11 15:38:51 UTC ---
> Need to check, but the patch also breaks bootstrap on ELF targets that
> lack stpcpy, like any Solaris < 10:

Correction: < 11.

    Rainer


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (2 preceding siblings ...)
  2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2011-11-11 16:01 ` dominiq at lps dot ens.fr
                   ` (25 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2011-11-11 15:41 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> 2011-11-11 15:35:14 UTC ---
> --- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-11 15:30:50 UTC ---
> Is it fixed now?

Need to check, but the patch also breaks bootstrap on ELF targets that
lack stpcpy, like any Solaris < 10:

/vol/gcc/src/hg/trunk/local/gcc/coverage.c: In function 'void
create_coverage()':
/vol/gcc/src/hg/trunk/local/gcc/coverage.c:1031:3: error: 'stpcpy' was not
declared in this scope
make[3]: *** [coverage.o] Error 1

    Rainer


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (3 preceding siblings ...)
  2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2011-11-11 16:01 ` dominiq at lps dot ens.fr
  2011-11-11 16:29 ` ro at CeBiTec dot Uni-Bielefeld.DE
                   ` (24 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: dominiq at lps dot ens.fr @ 2011-11-11 16:01 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #5 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2011-11-11 15:40:13 UTC ---
> Is it fixed now?

With a similar patch, I am now at stage2.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (4 preceding siblings ...)
  2011-11-11 16:01 ` dominiq at lps dot ens.fr
@ 2011-11-11 16:29 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2011-11-11 16:38 ` jason at gcc dot gnu.org
                   ` (23 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2011-11-11 16:29 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #6 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> 2011-11-11 15:46:33 UTC ---
And it breaks Solaris/SPARC bootstrap with both Sun as and gas.  See the
following testcase:

$ cat string.c
int
main (void)
{
  return 0;
}

* Wit Sun as, I get:

$ > ./xgcc -B./ -g -c string.c -save-temps
/usr/bin/as: "string.s", line 27: error: unknown opcode ".string"
/usr/bin/as: "string.s", line 27: error: statement syntax

  and many more.

* With gnu as, I get:

string.s: Assembler messages:
string.s:120: Error: can't resolve `.LLFB0' {*UND* section} - `.LLtext0' {.text
section}

  and several more.

    Rainer


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (5 preceding siblings ...)
  2011-11-11 16:29 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2011-11-11 16:38 ` jason at gcc dot gnu.org
  2011-11-11 23:32 ` jimis at gmx dot net
                   ` (22 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jason at gcc dot gnu.org @ 2011-11-11 16:38 UTC (permalink / raw)
  To: gcc-bugs

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |NEW

--- Comment #7 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-11 16:29:08 UTC ---
(In reply to comment #3)
> Need to check, but the patch also breaks bootstrap on ELF targets that
> lack stpcpy

Ah.  I thought that wouldn't be a problem because of the version in libiberty,
but I guess we still need a declaration.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (6 preceding siblings ...)
  2011-11-11 16:38 ` jason at gcc dot gnu.org
@ 2011-11-11 23:32 ` jimis at gmx dot net
  2011-11-11 23:40 ` jimis at gmx dot net
                   ` (21 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2011-11-11 23:32 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #8 from jimis <jimis at gmx dot net> 2011-11-11 23:00:47 UTC ---
Created attachment 25800
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25800
Use libiberty's stpcpy()

Does the attached patch solve the stpcpy() issue?


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (7 preceding siblings ...)
  2011-11-11 23:32 ` jimis at gmx dot net
@ 2011-11-11 23:40 ` jimis at gmx dot net
  2011-11-12  1:41 ` jimis at gmx dot net
                   ` (20 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2011-11-11 23:40 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #9 from jimis <jimis at gmx dot net> 2011-11-11 23:31:22 UTC ---
Created attachment 25801
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25801
fix for platforms not using GNU as

I redefined STRING_ASM_OP and changed ELF_STRING_LIMIT back to STRING_LIMIT.
Does this fix problems with Sun assembler?


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (8 preceding siblings ...)
  2011-11-11 23:40 ` jimis at gmx dot net
@ 2011-11-12  1:41 ` jimis at gmx dot net
  2011-11-12  3:20 ` jimis at gmx dot net
                   ` (19 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2011-11-12  1:41 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #10 from jimis <jimis at gmx dot net> 2011-11-11 23:34:55 UTC ---
Nevertheless, I'd prefer using ELF_STRING_LIMIT and ELF_STRING_ASM_OP names to
point out that they affect only elf_* functions. But this renaming would touch
various rare archs. Should I try it?


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (9 preceding siblings ...)
  2011-11-12  1:41 ` jimis at gmx dot net
@ 2011-11-12  3:20 ` jimis at gmx dot net
  2011-11-12 21:45 ` jason at gcc dot gnu.org
                   ` (18 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2011-11-12  3:20 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #11 from jimis <jimis at gmx dot net> 2011-11-11 23:38:54 UTC ---
Created attachment 25802
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25802
STRING_LIMIT and STRING_ASM_OP usage in gcc

See attached file for platforms that will need macro renaming.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (10 preceding siblings ...)
  2011-11-12  3:20 ` jimis at gmx dot net
@ 2011-11-12 21:45 ` jason at gcc dot gnu.org
  2011-11-12 22:04 ` jason at gcc dot gnu.org
                   ` (17 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jason at gcc dot gnu.org @ 2011-11-12 21:45 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #12 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-12 21:40:03 UTC ---
Author: jason
Date: Sat Nov 12 21:39:59 2011
New Revision: 181326

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181326
Log:
    PR bootstrap/51094
    * gcc/configure.ac: Check for stpcpy declaration.
    * system.h: Declare it if not.
    * config.in, configure: Regenerate.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config.in
    trunk/gcc/configure
    trunk/gcc/configure.ac
    trunk/gcc/system.h


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (11 preceding siblings ...)
  2011-11-12 21:45 ` jason at gcc dot gnu.org
@ 2011-11-12 22:04 ` jason at gcc dot gnu.org
  2011-11-12 22:16 ` jason at gcc dot gnu.org
                   ` (16 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jason at gcc dot gnu.org @ 2011-11-12 22:04 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #13 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-12 21:40:10 UTC ---
Author: jason
Date: Sat Nov 12 21:40:07 2011
New Revision: 181327

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181327
Log:
    PR bootstrap/51094
    * config/elfos.h (STRING_ASM_OP): Define.
    * varasm.c (default_elf_asm_output_limited_string): Use it.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/elfos.h
    trunk/gcc/varasm.c


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (12 preceding siblings ...)
  2011-11-12 22:04 ` jason at gcc dot gnu.org
@ 2011-11-12 22:16 ` jason at gcc dot gnu.org
  2011-11-14 11:54 ` iains at gcc dot gnu.org
                   ` (15 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jason at gcc dot gnu.org @ 2011-11-12 22:16 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #14 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-12 21:44:17 UTC ---
As you can see, I applied the stpcpy patch and the change to define/use
STRING_ASM_OP.  I don't have a strong opinion about which way to go with the
macro renaming.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (13 preceding siblings ...)
  2011-11-12 22:16 ` jason at gcc dot gnu.org
@ 2011-11-14 11:54 ` iains at gcc dot gnu.org
  2011-11-14 14:32 ` ro at CeBiTec dot Uni-Bielefeld.DE
                   ` (14 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: iains at gcc dot gnu.org @ 2011-11-14 11:54 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #15 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-14 08:47:16 UTC ---
Created attachment 25814
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25814
adjustment to final.c

(just a heads up)
although not a bootstrap issue, this revision also caused a fair number of
regressions at m64 on i686/powerpc-darwin9.  This is where the m32 target has
an m64 multilib.

I am testing the patch on *-darwin9 and x86-64-darwin10.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (14 preceding siblings ...)
  2011-11-14 11:54 ` iains at gcc dot gnu.org
@ 2011-11-14 14:32 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2011-11-14 17:02 ` jason at gcc dot gnu.org
                   ` (13 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2011-11-14 14:32 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #16 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> 2011-11-14 14:17:50 UTC ---
> --- Comment #15 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-14 08:47:16 UTC ---
> Created attachment 25814
>   --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25814
> adjustment to final.c
>
> (just a heads up)
> although not a bootstrap issue, this revision also caused a fair number of
> regressions at m64 on i686/powerpc-darwin9.  This is where the m32 target has
> an m64 multilib.
>
> I am testing the patch on *-darwin9 and x86-64-darwin10.

I'm seeing the same issue on i386-pc-solaris2.11 for the 64-bit
multilib, while amd64-pc-solaris2.11 is fine.

I'm running a bootstrap with your patch included on i386-pc-solaris2.10
now.

On Solaris/SPARC, I also need the patch for PR rtl-optimization/51051
and remove the definition of ASM_GENERATE_INTERNAL_LABEL in
gcc/config/sparc/sol2.h.  There's going to be much similar fallout in
other ports, too, since private definitions of that macro are widespread.

Thanks.
        Rainer


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (15 preceding siblings ...)
  2011-11-14 14:32 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2011-11-14 17:02 ` jason at gcc dot gnu.org
  2011-11-15 10:03 ` iains at gcc dot gnu.org
                   ` (12 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jason at gcc dot gnu.org @ 2011-11-14 17:02 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #17 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-14 16:56:07 UTC ---
Author: jason
Date: Mon Nov 14 16:55:56 2011
New Revision: 181355

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181355
Log:
    PR bootstrap/51094
    PR middle-end/51116
    * config/elfos.h (TARGET_ASM_INTERNAL_LABEL): Remove.
    * varasm.c (default_elf_internal_label): Remove.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/elfos.h
    trunk/gcc/varasm.c


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (16 preceding siblings ...)
  2011-11-14 17:02 ` jason at gcc dot gnu.org
@ 2011-11-15 10:03 ` iains at gcc dot gnu.org
  2011-11-17 17:51 ` ro at CeBiTec dot Uni-Bielefeld.DE
                   ` (11 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: iains at gcc dot gnu.org @ 2011-11-15 10:03 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #18 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-15 09:34:00 UTC ---
(In reply to comment #16)
> > --- Comment #15 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-14 08:47:16 UTC ---
> > Created attachment 25814 [details]
> >   --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25814
> > adjustment to final.c
> >
> > (just a heads up)
> > although not a bootstrap issue, this revision also caused a fair number of
> > regressions at m64 on i686/powerpc-darwin9.  This is where the m32 target has
> > an m64 multilib.
> >
> > I am testing the patch on *-darwin9 and x86-64-darwin10.

The patch fixes those particular fails for me on *-darwin9, 
if it works for you then should it be posted - or does Jason want to amend?


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (17 preceding siblings ...)
  2011-11-15 10:03 ` iains at gcc dot gnu.org
@ 2011-11-17 17:51 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2011-11-17 18:02 ` iains at gcc dot gnu.org
                   ` (10 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2011-11-17 17:51 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #19 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> 2011-11-17 17:40:02 UTC ---
> --- Comment #18 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-15 09:34:00 UTC ---
> (In reply to comment #16)
>> > --- Comment #15 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-14 08:47:16 UTC ---
>> > Created attachment 25814 [details]
>> >   --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25814
>> > adjustment to final.c
>> >
>> > (just a heads up)
>> > although not a bootstrap issue, this revision also caused a fair number of
>> > regressions at m64 on i686/powerpc-darwin9.  This is where the m32 target has
>> > an m64 multilib.
>> >
>> > I am testing the patch on *-darwin9 and x86-64-darwin10.
>
> The patch fixes those particular fails for me on *-darwin9, 
> if it works for you then should it be posted - or does Jason want to amend?

Indeed: the patch is still necessary to get sort-of decent testsuite
results on i386-pc-solaris2.1[01].

    Rainer


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (18 preceding siblings ...)
  2011-11-17 17:51 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2011-11-17 18:02 ` iains at gcc dot gnu.org
  2011-12-05 14:20 ` rguenth at gcc dot gnu.org
                   ` (9 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: iains at gcc dot gnu.org @ 2011-11-17 18:02 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #20 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-17 17:50:09 UTC ---
(In reply to comment #19)
> > --- Comment #18 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-15 09:34:00 UTC ---
> > (In reply to comment #16)
> >> > --- Comment #15 from Iain Sandoe <iains at gcc dot gnu.org> 2011-11-14 08:47:16 UTC ---
> >> > Created attachment 25814 [details]
> >> >   --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25814
> >> > adjustment to final.c
> >> >
> >> > (just a heads up)
> >> > although not a bootstrap issue, this revision also caused a fair number of
> >> > regressions at m64 on i686/powerpc-darwin9.  This is where the m32 target has
> >> > an m64 multilib.
> >> >
> >> > I am testing the patch on *-darwin9 and x86-64-darwin10.
> >
> > The patch fixes those particular fails for me on *-darwin9, 
> > if it works for you then should it be posted - or does Jason want to amend?
> 
> Indeed: the patch is still necessary to get sort-of decent testsuite
> results on i386-pc-solaris2.1[01].

I think the patch needs a tweak -
- the case of largest negative int in the case where HOST_WIDE_INT is long
long.

I wonder if it's simply OK to bail to the original printf for that corner-case,
or whether it's worth handling specially.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (19 preceding siblings ...)
  2011-11-17 18:02 ` iains at gcc dot gnu.org
@ 2011-12-05 14:20 ` rguenth at gcc dot gnu.org
  2011-12-05 15:02 ` dominiq at lps dot ens.fr
                   ` (8 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-12-05 14:20 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.0


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (20 preceding siblings ...)
  2011-12-05 14:20 ` rguenth at gcc dot gnu.org
@ 2011-12-05 15:02 ` dominiq at lps dot ens.fr
  2011-12-06 13:58 ` rguenth at gcc dot gnu.org
                   ` (7 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: dominiq at lps dot ens.fr @ 2011-12-05 15:02 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #21 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2011-12-05 15:01:49 UTC ---
There is no longer any bootstrap issues. Does it remain something to fix?
Otherwise I'll close this PR as fixed.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (21 preceding siblings ...)
  2011-12-05 15:02 ` dominiq at lps dot ens.fr
@ 2011-12-06 13:58 ` rguenth at gcc dot gnu.org
  2012-07-08 10:02 ` jimis at gmx dot net
                   ` (6 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: rguenth at gcc dot gnu.org @ 2011-12-06 13:58 UTC (permalink / raw)
  To: gcc-bugs

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

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

--- Comment #22 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-12-06 13:58:08 UTC ---
Bootstrap issue fixed.  Open a new bug for anything that is left.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (22 preceding siblings ...)
  2011-12-06 13:58 ` rguenth at gcc dot gnu.org
@ 2012-07-08 10:02 ` jimis at gmx dot net
  2012-07-08 10:25 ` iains at gcc dot gnu.org
                   ` (5 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2012-07-08 10:02 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #23 from jimis <jimis at gmx dot net> 2012-07-08 10:02:14 UTC ---
Iain, ro: Do you know of some darwin or solaris i386 machine where I can test
some changes to this patch? 

Does the bug mentioned in comment #15 and comment #16 needs special configure
flags? Maybe I can reproduce it on x86 somehow? (how to build 64-bit multilib
on 32 bit host?)


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (23 preceding siblings ...)
  2012-07-08 10:02 ` jimis at gmx dot net
@ 2012-07-08 10:25 ` iains at gcc dot gnu.org
  2012-07-09 10:04 ` jimis at gmx dot net
                   ` (4 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: iains at gcc dot gnu.org @ 2012-07-08 10:25 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #24 from Iain Sandoe <iains at gcc dot gnu.org> 2012-07-08 10:24:45 UTC ---
(In reply to comment #23)
> Iain, ro: Do you know of some darwin or solaris i386 machine where I can test
> some changes to this patch? 
> 
> Does the bug mentioned in comment #15 and comment #16 needs special configure
> flags? Maybe I can reproduce it on x86 somehow? (how to build 64-bit multilib
> on 32 bit host?)

If you want a patch testing - then I'm sure that Dominique, me or Jack can test
on Darwin.  Unfortunately, I am not in a position to give remote access to my
Darwin boxen.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (24 preceding siblings ...)
  2012-07-08 10:25 ` iains at gcc dot gnu.org
@ 2012-07-09 10:04 ` jimis at gmx dot net
  2012-07-09 10:07 ` jimis at gmx dot net
                   ` (3 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2012-07-09 10:04 UTC (permalink / raw)
  To: gcc-bugs

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

jimis <jimis at gmx dot net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |abel at gcc dot gnu.org

--- Comment #25 from jimis <jimis at gmx dot net> 2012-07-09 10:03:37 UTC ---
I attach a patch almost similar to yours that I think handles the max negative
issue (but presumes 2's complement representation, and that HOST_WIDE_INT is
always larger or equal type than long). 

Passed testing on i386, bootstrapped fine on x86_64 multilib, I'd appreciate
testing on 32-bit multilib platform.


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (25 preceding siblings ...)
  2012-07-09 10:04 ` jimis at gmx dot net
@ 2012-07-09 10:07 ` jimis at gmx dot net
  2012-07-09 16:28 ` ro at CeBiTec dot Uni-Bielefeld.DE
                   ` (2 subsequent siblings)
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2012-07-09 10:07 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #26 from jimis <jimis at gmx dot net> 2012-07-09 10:06:53 UTC ---
Created attachment 27765
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27765
fprint_w reinstated


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (26 preceding siblings ...)
  2012-07-09 10:07 ` jimis at gmx dot net
@ 2012-07-09 16:28 ` ro at CeBiTec dot Uni-Bielefeld.DE
  2012-07-09 17:37 ` iains at gcc dot gnu.org
  2012-07-09 19:57 ` jimis at gmx dot net
  29 siblings, 0 replies; 31+ messages in thread
From: ro at CeBiTec dot Uni-Bielefeld.DE @ 2012-07-09 16:28 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #27 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot Uni-Bielefeld.DE> 2012-07-09 16:28:24 UTC ---
> Passed testing on i386, bootstrapped fine on x86_64 multilib, I'd appreciate
> testing on 32-bit multilib platform.

I've just completed a regtest with your patch on i386-pc-solaris2.10
(both 32 and 64-bit multilibs) without regressions.

    Rainer


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (27 preceding siblings ...)
  2012-07-09 16:28 ` ro at CeBiTec dot Uni-Bielefeld.DE
@ 2012-07-09 17:37 ` iains at gcc dot gnu.org
  2012-07-09 19:57 ` jimis at gmx dot net
  29 siblings, 0 replies; 31+ messages in thread
From: iains at gcc dot gnu.org @ 2012-07-09 17:37 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #28 from Iain Sandoe <iains at gcc dot gnu.org> 2012-07-09 17:37:11 UTC ---
(In reply to comment #27)
> > Passed testing on i386, bootstrapped fine on x86_64 multilib, I'd appreciate
> > testing on 32-bit multilib platform.
> 
> I've just completed a regtest with your patch on i386-pc-solaris2.10
> (both 32 and 64-bit multilibs) without regressions.

likewise for i686-darwin9


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

* [Bug bootstrap/51094] [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets
  2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
                   ` (28 preceding siblings ...)
  2012-07-09 17:37 ` iains at gcc dot gnu.org
@ 2012-07-09 19:57 ` jimis at gmx dot net
  29 siblings, 0 replies; 31+ messages in thread
From: jimis at gmx dot net @ 2012-07-09 19:57 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #29 from jimis <jimis at gmx dot net> 2012-07-09 19:57:22 UTC ---
Thanks guys, sent to gcc-patches:

http://gcc.gnu.org/ml/gcc-patches/2012-07/msg00345.html


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

end of thread, other threads:[~2012-07-09 19:57 UTC | newest]

Thread overview: 31+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-11-11 13:13 [Bug bootstrap/51094] New: [4.7 Regression] Bootstrap failure at revision 181279 on non-ELF targets dominiq at lps dot ens.fr
2011-11-11 13:19 ` [Bug bootstrap/51094] " ro at gcc dot gnu.org
2011-11-11 15:39 ` jason at gcc dot gnu.org
2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
2011-11-11 15:41 ` ro at CeBiTec dot Uni-Bielefeld.DE
2011-11-11 16:01 ` dominiq at lps dot ens.fr
2011-11-11 16:29 ` ro at CeBiTec dot Uni-Bielefeld.DE
2011-11-11 16:38 ` jason at gcc dot gnu.org
2011-11-11 23:32 ` jimis at gmx dot net
2011-11-11 23:40 ` jimis at gmx dot net
2011-11-12  1:41 ` jimis at gmx dot net
2011-11-12  3:20 ` jimis at gmx dot net
2011-11-12 21:45 ` jason at gcc dot gnu.org
2011-11-12 22:04 ` jason at gcc dot gnu.org
2011-11-12 22:16 ` jason at gcc dot gnu.org
2011-11-14 11:54 ` iains at gcc dot gnu.org
2011-11-14 14:32 ` ro at CeBiTec dot Uni-Bielefeld.DE
2011-11-14 17:02 ` jason at gcc dot gnu.org
2011-11-15 10:03 ` iains at gcc dot gnu.org
2011-11-17 17:51 ` ro at CeBiTec dot Uni-Bielefeld.DE
2011-11-17 18:02 ` iains at gcc dot gnu.org
2011-12-05 14:20 ` rguenth at gcc dot gnu.org
2011-12-05 15:02 ` dominiq at lps dot ens.fr
2011-12-06 13:58 ` rguenth at gcc dot gnu.org
2012-07-08 10:02 ` jimis at gmx dot net
2012-07-08 10:25 ` iains at gcc dot gnu.org
2012-07-09 10:04 ` jimis at gmx dot net
2012-07-09 10:07 ` jimis at gmx dot net
2012-07-09 16:28 ` ro at CeBiTec dot Uni-Bielefeld.DE
2012-07-09 17:37 ` iains at gcc dot gnu.org
2012-07-09 19:57 ` jimis at gmx dot net

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).