[Bug target/44732] New: [4.6 Regression] bootstrap broken on ia64

[Bug target/44732] New: [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

During stage1:

gcc -c  -DUSE_LIBUNWIND_EXCEPTIONS  -g -fkeep-inline-functions -DIN_GCC   -W
-Wall -Wwrite-strings -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
-Wmissing-format-attribute -Wold-style-definition -Wc++-compat -fno-common 
-DHAVE_CONFIG_H -I. -I. -I../../trunk/gcc -I../../trunk/gcc/.
-I../../trunk/gcc/../include -I../../trunk/gcc/../libcpp/include 
-I../../trunk/gcc/../libdecnumber -I../../trunk/gcc/../libdecnumber/dpd
-I../libdecnumber   -I/usr/include/libelf  \
        ../../trunk/gcc/config/ia64/ia64.c -o ia64.o
../../trunk/gcc/config/ia64/ia64.c:559: warning: initialization from
incompatible pointer type
../../trunk/gcc/config/ia64/ia64.c: In function ?ia64_handle_model_attribute?:
../../trunk/gcc/config/ia64/ia64.c:618: warning: unknown conversion type
character ?E? in format
../../trunk/gcc/config/ia64/ia64.c:618: warning: too many arguments for format
../../trunk/gcc/config/ia64/ia64.c:652: warning: unknown conversion type
character ?E? in format
../../trunk/gcc/config/ia64/ia64.c:652: warning: too many arguments for format
../../trunk/gcc/config/ia64/ia64.c: In function
?ia64_vms_common_object_attribute?:
../../trunk/gcc/config/ia64/ia64.c:686: warning: unknown conversion type
character ?E? in format
../../trunk/gcc/config/ia64/ia64.c:686: warning: too many arguments for format
../../trunk/gcc/config/ia64/ia64.c: In function ?ia64_register_move_cost?:
../../trunk/gcc/config/ia64/ia64.c:5286: warning: no previous prototype for
?ia64_preferred_reload_class?
../../trunk/gcc/config/ia64/ia64.c:5323: warning: no previous prototype for
?ia64_secondary_reload_class?
../../trunk/gcc/config/ia64/ia64.c:5412: error: invalid storage class for
function ?ia64_unspec_may_trap_p?
../../trunk/gcc/config/ia64/ia64.c:5412: warning: no previous prototype for
?ia64_unspec_may_trap_p?
../../trunk/gcc/config/ia64/ia64.c:5436: error: invalid storage class for
function ?fix_range?
../../trunk/gcc/config/ia64/ia64.c:5436: warning: no previous prototype for
?fix_range?
../../trunk/gcc/config/ia64/ia64.c:5501: error: invalid storage class for
function ?ia64_handle_option?
../../trunk/gcc/config/ia64/ia64.c:5501: warning: no previous prototype for
?ia64_handle_option?
../../trunk/gcc/config/ia64/ia64.c:5548: warning: no previous prototype for
?ia64_override_options?
../../trunk/gcc/config/ia64/ia64.c:5578: error: invalid storage class for
function ?ia64_override_options_after_change?
...


-- 
           Summary: [4.6 Regression] bootstrap broken on ia64
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Keywords: build
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rguenth at gcc dot gnu dot org
GCC target triplet: ia64-gnu-linux


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2010-06-30 20:31 -------
r161633, 4.1 host compiler.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.0


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2010-07-01 08:17 -------
*** Bug 44740 has been marked as a duplicate of this bug. ***


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl dot tools at gmail dot
                   |                            |com


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2010-07-01 08:34 -------
r161633 is the cause.


-- 

rguenth at gcc dot gnu dot org changed:

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


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[amylaar at gcc dot gnu dot org]

------- Comment #4 from amylaar at gcc dot gnu dot org  2010-07-01 09:44 -------
What is the target triple from config.guess?  I've tried cross compilers
from i686-pc-linux-gnu to ia64-elf and ia64-linux-gnu
(using ccc (GCC) 4.4.4 20100503 (Red Hat 4.4.4-2)), but I get lots
of extra errors (and even more warnings).

ia64-elf:
[amylaar@laria gcc]$ make ia64.o 2>&1 | grep -c error
128
[amylaar@laria gcc]$ ^C
[amylaar@laria gcc]$ make ia64.o 2>&1 | wc
    414    3038   40231
[amylaar@laria gcc]$ make ia64.o 2>&1 | grep -c error:
128
ia64-linux-gnu:
[amylaar@laria gcc]$ make ia64.o 2>&1|wc
    414    3039   40258
[amylaar@laria gcc]$ make ia64.o 2>&1|grep -c error:
128


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenther at suse dot de]

------- Comment #5 from rguenther at suse dot de  2010-07-01 09:54 -------
Subject: Re:  [4.6 Regression] bootstrap broken on ia64

On Thu, 1 Jul 2010, amylaar at gcc dot gnu dot org wrote:

> ------- Comment #4 from amylaar at gcc dot gnu dot org  2010-07-01 09:44 -------
> What is the target triple from config.guess?  I've tried cross compilers
> from i686-pc-linux-gnu to ia64-elf and ia64-linux-gnu
> (using ccc (GCC) 4.4.4 20100503 (Red Hat 4.4.4-2)), but I get lots
> of extra errors (and even more warnings).
> 
> ia64-elf:
> [amylaar@laria gcc]$ make ia64.o 2>&1 | grep -c error
> 128
> [amylaar@laria gcc]$ ^C
> [amylaar@laria gcc]$ make ia64.o 2>&1 | wc
>     414    3038   40231
> [amylaar@laria gcc]$ make ia64.o 2>&1 | grep -c error:
> 128
> ia64-linux-gnu:
> [amylaar@laria gcc]$ make ia64.o 2>&1|wc
>     414    3039   40258
> [amylaar@laria gcc]$ make ia64.o 2>&1|grep -c error:
> 128

I see the issue with the host compiler, so it might be a host
issue (using GCC 4.1).  The target triple is ia64-unknown-linux-gnu.

Richard.


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

------- Comment #6 from rguenth at gcc dot gnu dot org  2010-07-01 10:02 -------
Created an attachment (id=21054)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21054&action=view)
preprocessed source

Preprocessed source of ia64.i for your convenience.


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[amylaar at gcc dot gnu dot org]

------- Comment #7 from amylaar at gcc dot gnu dot org  2010-07-01 10:27 -------
Subject: Bug 44732

Author: amylaar
Date: Thu Jul  1 10:27:34 2010
New Revision: 161658

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161658
Log:
        PR target/44732
        * config/ia64/ia64.c (ia64_register_move_cost): Remove stray '{'.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/ia64/ia64.c


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[amylaar at gcc dot gnu dot org]

------- Comment #8 from amylaar at gcc dot gnu dot org  2010-07-01 10:35 -------
(In reply to comment #6)
> Created an attachment (id=21054)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21054&action=view) [edit]
> preprocessed source
> 
> Preprocessed source of ia64.i for your convenience.

Thanks, in the meantime, I found out that all these errors are genuine - sort
of.  The real smoking gun are 'nested function' warnings.  By searching for
the first one, I found a stray '{'.  I've committed the fix as obvious.


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenther at suse dot de]

------- Comment #9 from rguenther at suse dot de  2010-07-01 10:37 -------
Subject: Re:  [4.6 Regression] bootstrap broken on ia64

On Thu, 1 Jul 2010, amylaar at gcc dot gnu dot org wrote:

> ------- Comment #8 from amylaar at gcc dot gnu dot org  2010-07-01 10:35 -------
> (In reply to comment #6)
> > Created an attachment (id=21054)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21054&action=view)
>  --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21054&action=view) [edit]
> > preprocessed source
> > 
> > Preprocessed source of ia64.i for your convenience.
> 
> Thanks, in the meantime, I found out that all these errors are genuine - sort
> of.  The real smoking gun are 'nested function' warnings.  By searching for
> the first one, I found a stray '{'.  I've committed the fix as obvious.

Thanks for the quick fix.  I'm checking if bootstrap is fixed now.


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[amylaar at gcc dot gnu dot org]

------- Comment #10 from amylaar at gcc dot gnu dot org  2010-07-01 10:49 -------
As I used cut & paste for a few repeated stanzas, I just checked if
there was any more of the same.
Once you know what it is, it's easy to find in the patch:

[amylaar@laria ~]$ grep -A 5 -B 5 '^+[  ]*{' 44566-regclass-diff-3
 static int
-ia64_register_move_cost (enum machine_mode mode, enum reg_class from,
-                        enum reg_class to)
+ia64_register_move_cost (enum machine_mode mode, enum reg_class from_i,
+                        enum reg_class to_i)
+{
+  enum reg_class from = (enum reg_class) from_i;
+  enum reg_class to = (enum reg_class) to_i;
 {
   /* ADDL_REGS is the same as GR_REGS for movement purposes.  */
   if (to == ADDL_REGS)

so this typo was specific to ia64.c


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1851 bytes --]



------- Comment #11 from rguenth at gcc dot gnu dot org  2010-07-01 11:02 -------
Now we're back to

/gcc/obj/./prev-gcc/xgcc -B/gcc/obj/./prev-gcc/
-B/usr/local/ia64-unknown-linux-gnu/bin/
-B/usr/local/ia64-unknown-linux-gnu/bin/
-B/usr/local/ia64-unknown-linux-gnu/lib/ -isystem
/usr/local/ia64-unknown-linux-gnu/include -isystem
/usr/local/ia64-unknown-linux-gnu/sys-include    -c  -DUSE_LIBUNWIND_EXCEPTIONS
 -g -O2 -gtoggle -DIN_GCC   -W -Wall -Wwrite-strings -Wcast-qual
-Wstrict-prototypes -Wmissing-prototypes -Wmissing-format-attribute -pedantic
-Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -Werror
-Wold-style-definition -Wc++-compat -fno-common  -DHAVE_CONFIG_H -I. -I.
-I../../trunk/gcc -I../../trunk/gcc/. -I../../trunk/gcc/../include
-I../../trunk/gcc/../libcpp/include  -I../../trunk/gcc/../libdecnumber
-I../../trunk/gcc/../libdecnumber/dpd -I../libdecnumber   -I/usr/include/libelf
 \
        ../../trunk/gcc/config/ia64/ia64.c -o ia64.o
../../trunk/gcc/config/ia64/ia64.c:559:8: error: initialization from
incompatible pointer type [-Werror]
../../trunk/gcc/config/ia64/ia64.c:559:8: error: (near initialization for
‘targetm.register_move_cost’) [-Werror]
cc1: all warnings being treated as errors

which is I think what HJ reported originally in PR44740.

This might or might not be your change again (the target.def one).

Thus, bootstrap is still broken.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-07-01 11:02:47
               date|                            |


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[amylaar at gcc dot gnu dot org]

------- Comment #12 from amylaar at gcc dot gnu dot org  2010-07-01 11:18 -------
Created an attachment (id=21056)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21056&action=view)
        * ia64_register_move_cost (Fix argument types).

Yes, it was my patch.
With the attached patch, yesterday's gcc 4.6.0 accepts ia64.c without warning:

[amylaar@laria gcc]$ /user/inria/20100630/bin/gcc -c 
-DUSE_LIBUNWIND_EXCEPTIONS  -g  -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE  -W -Wall
-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../../gcc/gcc -I../../gcc/gcc/.
-I../../gcc/gcc/../include -I../../gcc/gcc/../libcpp/include 
-I../../gcc/gcc/../libdecnumber -I../../gcc/gcc/../libdecnumber/dpd
-I../libdecnumber  -DCLOOG_PPL_BACKEND  -I/usr/include/libelf                 
../../gcc/gcc/config/ia64/ia64.c -o ia64.o
[amylaar@laria gcc]$ 


-- 


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

[Bug target/44732] [4.6 Regression] bootstrap broken on ia64

[rguenth at gcc dot gnu dot org]

------- Comment #13 from rguenth at gcc dot gnu dot org  2010-07-01 15:51 -------
Bootstrap finished fine for me.


-- 

rguenth at gcc dot gnu dot org changed:

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


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