[Bug target/24934] New: profilebootstrap failure with debugging disabled

[Bug target/24934] New: profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

profilebootstrap with CFLAGS="-O2" BOOT_CFLAGS="-O2" fails with

/tmp/ccloJJLt.s: Assembler messages:
/tmp/ccloJJLt.s:138: Warning: Corrupted unwind info due to unsupported section
switching
/tmp/ccloJJLt.s:747: Internal error!
Assertion failure in ia64_estimate_size_before_relax at
../../gas/config/tc-ia64.c line 3098.
Please report this bug.

This is related to pr22313, but an extended patch does not fix this problem.


-- 
           Summary: profilebootstrap failure with debugging disabled
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Keywords: wrong-code
          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-unknown-linux-gnu


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

[Bug target/24934] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2005-11-18 20:17 -------
Created an attachment (id=10278)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10278&action=view)
testcase

Testcase


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2005-11-18 20:18 -------
Created an attachment (id=10279)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10279&action=view)
profile data 1/2


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
  BugsThisDependsOn|                            |22313
            Summary|profilebootstrap failure    |[4.1 Regression]
                   |with debugging disabled     |profilebootstrap failure
                   |                            |with debugging disabled
   Target Milestone|---                         |4.1.0


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #3 from rguenth at gcc dot gnu dot org  2005-11-18 20:19 -------
Created an attachment (id=10280)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10280&action=view)
profile data 2/2


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #4 from rguenth at gcc dot gnu dot org  2005-11-18 20:28 -------
Grrr, the testcase was created with some older binutils.  Trying to create one
with one that matches the reported error.


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #6 from rguenth at gcc dot gnu dot org  2005-11-18 20:59 -------
Created an attachment (id=10283)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10283&action=view)
profile data 1/2


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #10279|0                           |1
        is obsolete|                            |


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #5 from rguenth at gcc dot gnu dot org  2005-11-18 20:59 -------
Created an attachment (id=10282)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10282&action=view)
testcase

new testcase.  Compile with

stage1/cc1 -fpreprocessed c-lex.i -quiet -dumpbase c-lex.i -auxbase-strip
c-lex.o -O2 -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes
-pedantic -Wno-long-long -Wno-variadic-macros -Wold-style-definition
-Wmissing-format-attribute -Werror -version -fprofile-use
-freorder-blocks-and-partition -fno-common -o /tmp/cciU35Gj.s
GNU C version 4.1.0 20051116 (experimental) (SUSE Linux)
(ia64-unknown-linux-gnu)
        compiled by GNU C version 4.1.0 20051116 (experimental) (SUSE Linux).
GGC heuristics: --param ggc-min-expand=30 --param ggc-min-heapsize=4096
Compiler executable checksum: 1c9d4c0e30064cb5ef51bb1a6548d214
 stage1/as -x -o c-lex.o /tmp/cciU35Gj.s
/tmp/cciU35Gj.s: Assembler messages:
/tmp/cciU35Gj.s:308: Warning: Corrupted unwind info due to unsupported section
switching
/tmp/cciU35Gj.s:4760: Internal error!
Assertion failure in ia64_estimate_size_before_relax at
../../gas/config/tc-ia64.c line 3098.
Please report this bug.

not using -freorder-blocks-and-partition fixes the problem.  Adding -g
does not make a difference.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #10278|0                           |1
        is obsolete|                            |


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #7 from rguenth at gcc dot gnu dot org  2005-11-18 21:00 -------
Created an attachment (id=10284)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10284&action=view)
profile data 2/2


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #10280|0                           |1
        is obsolete|                            |


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #8 from rguenth at gcc dot gnu dot org  2005-11-18 21:00 -------
Created an attachment (id=10285)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10285&action=view)
wrong assembly


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #9 from rguenth at gcc dot gnu dot org  2005-11-18 21:01 -------
Arguably an as bug, but maybe we should not use -freorder-blocks-and-partition
on ia64 profiledbootstrap.


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[pinskia at gcc dot gnu dot org]

------- Comment #10 from pinskia at gcc dot gnu dot org  2005-11-18 21:02 -------
So if -g does not fix it, then this is not related at all to PR 22313.
We should just disable --freorder-blocks-and-partition if gas is unwilling to
support unwind info when the section changes.

See http://sources.redhat.com/bugzilla/show_bug.cgi?id=994 for when the warning
was introduced and more information on this bug.


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[rguenth at gcc dot gnu dot org]

------- Comment #11 from rguenth at gcc dot gnu dot org  2005-11-18 21:07 -------
See http://sources.redhat.com/bugzilla/show_bug.cgi?id=994 for the binutils
bug.


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure with debugging disabled

[mmitchel at gcc dot gnu dot org]

------- Comment #12 from mmitchel at gcc dot gnu dot org  2005-11-19 02:22 -------
This is too obscure to be release-critical.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P5


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure

[rguenth at gcc dot gnu dot org]

------- Comment #13 from rguenth at gcc dot gnu dot org  2005-11-19 10:38 -------
Changing the summary to reflect reality and remove some of the obscure-ness. 
Mark, what was the obscureness you are refering to?


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mmitchel at gcc dot gnu dot
                   |                            |org
            Summary|[4.1 Regression]            |[4.1 Regression]
                   |profilebootstrap failure    |profilebootstrap failure
                   |with debugging disabled     |


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure

[pinskia at gcc dot gnu dot org]

------- Comment #14 from pinskia at gcc dot gnu dot org  2005-11-19 16:22 -------
(In reply to comment #13)
> Changing the summary to reflect reality and remove some of the obscure-ness. 
> Mark, what was the obscureness you are refering to?

well both using BOOT_CFLAGS and profiledbootstrap together is less likely than
just bootstrap and using BOOT_CFLAGS.


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure

[mark at codesourcery dot com]

------- Comment #15 from mark at codesourcery dot com  2005-11-19 17:34 -------
Subject: Re:  [4.1 Regression] profilebootstrap failure

rguenth at gcc dot gnu dot org wrote:
> ------- Comment #13 from rguenth at gcc dot gnu dot org  2005-11-19 10:38 -------
> Changing the summary to reflect reality and remove some of the obscure-ness. 
> Mark, what was the obscureness you are refering to?

Sorry, that was indeed unclear.

I don't consider building the compiler with profiledbootstrap to be a
fundamental usage model.  Most people don't do it, and there's an easy
work-around: build with normal bootstrap.


-- 


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

[Bug target/24934] [4.1 Regression] profilebootstrap failure

[pinskia at gcc dot gnu dot org]

------- Comment #16 from pinskia at gcc dot gnu dot org  2005-11-20 00:38 -------
Confirmed.


-- 

pinskia 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         |2005-11-20 00:38:48
               date|                            |


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

[Bug target/24934] [4.1/4.2 Regression] profilebootstrap failure

[wilson at gcc dot gnu dot org]

------- Comment #17 from wilson at gcc dot gnu dot org  2005-11-23 00:16 -------
The hot/cold section partitioning stuff has no support for generating unwind
info.

See for instance in opts.c where this is turned off when flag_exceptions is
true, which means it is always turned off by default for C++.  This is because
C++ requires unwind info for the EH support, and we can't generate correct
unwind info when host/cold section partitioning is on.  This is true for all
targets.

The reason why there is an IA-64 issue here is because unwind info is a
required part of the ABI.  We must always generate unwind info, even for C, and
thus this option is never safe on IA-64.  But it will work for all other
targets as they don't require unwind info for C code.

I believe the correct fix here is to turn off the hot/cold partitioning support
when flag_unwind_tables is defined.  This is turned on by default in the IA-64
backend.  This can also be turned on by the user.

There are also some peripheral issues here.  We should reconsider whether
profiledbootstrap should include an option that does not support all targets
and all languages.  This could be trouble later.  Also, the IA-64 assembler
should not be core dumping here.


-- 

wilson at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |wilson at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2005-11-20 00:38:48         |2005-11-23 00:16:39
               date|                            |


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

[Bug target/24934] [4.1/4.2 Regression] profilebootstrap failure

[wilson at gcc dot gnu dot org]

------- Comment #18 from wilson at gcc dot gnu dot org  2005-11-23 00:20 -------
Created an attachment (id=10320)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10320&action=view)
Turn off partitioning optimization when flag_unwind_tables true.

Suggested untested patch.  Gives different message for user requested unwind
info versus target requested unwind info.  This part is optional, and only
affects IA-64 when flag_unwind_tables is the default.


-- 


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

[Bug target/24934] [4.1/4.2 Regression] profilebootstrap failure

[wilson at gcc dot gnu dot org]

------- Comment #19 from wilson at gcc dot gnu dot org  2005-12-06 05:31 -------
Subject: Bug 24934

Author: wilson
Date: Tue Dec  6 05:31:39 2005
New Revision: 108103

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=108103
Log:
PR target/24934
* opts.c (decode_options): Turn off partitioning if flag_unwind_tables
is set.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/opts.c


-- 


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

[Bug target/24934] [4.1/4.2 Regression] profilebootstrap failure

[wilson at gcc dot gnu dot org]

------- Comment #20 from wilson at gcc dot gnu dot org  2005-12-06 05:41 -------
Subject: Bug 24934

Author: wilson
Date: Tue Dec  6 05:41:33 2005
New Revision: 108104

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=108104
Log:
PR target/24934
* opts.c (decode_options): Turn off partitioning if flag_unwind_tables
is set.

Modified:
    branches/gcc-4_1-branch/gcc/ChangeLog
    branches/gcc-4_1-branch/gcc/opts.c


-- 


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

[Bug target/24934] [4.1/4.2 Regression] profilebootstrap failure

[wilson at gcc dot gnu dot org]

------- Comment #21 from wilson at gcc dot gnu dot org  2005-12-06 05:55 -------
Fixed on mainline and the gcc-4.1 branch.


-- 

wilson at gcc dot gnu dot org changed:

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


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