[Bug target/20286] [3.3 only] gcc panic with __thread attribute

[wilson at gcc dot gnu dot org]

--- Additional Comments From wilson at gcc dot gnu dot org  2005-03-24 
00:09 ---
That patch that fixed this in gcc-3.4 is too large to backport, particularly
when considering that it depends on other stuff (such as
default_encode_section_info) which is not already present in gcc-3.3.  However,
it can be fixed with a simpler patch.

The underlying problem is that a variable changes state from
TLS_MODEL_INTIAL_EXEC (extern) to TLS_MODEL_LOCAL_EXEC (static).  This is an
allowable transition.  This unfortunately triggers an abort in the ia64.c
backend in ia64_encode_section_info.  The comment next to the abort makes it
clear that this is checking for transitions  between thread data and small data.
 So the fix is to tighten the conditions under which we call the abort to match
the conditions documented in the comments.

The same INITIAL_EXEC to LOCAL_EXEC transition happens in gcc-3.4, there just
isn't a corresponding abort in default_encode_section_info.

Technically, this isn't a regression, as gcc-3.2 did not have TLS support, and
gcc-3.3 is only open for regression fixes, so there is the question of whether
this patch is OK to install on the branch.  A strict interpretation of the rules
says this is not OK, at least not without approval of the branch maintainer, 
Gaby.

-- 
   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |wilson at gcc dot gnu dot
   |dot org |org
 Status|NEW |ASSIGNED
   Last reconfirmed|2005-03-02 16:48:16 |2005-03-24 00:09:55
   date||


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

[Bug target/20286] [3.3 only] gcc panic with __thread attribute

[gdr at integrable-solutions dot net]

--- Additional Comments From gdr at integrable-solutions dot net  
2005-03-25 22:13 ---
Subject: Re:  [3.3 only] gcc panic with __thread attribute

"wilson at gcc dot gnu dot org" <[EMAIL PROTECTED]> writes:

| Technically, this isn't a regression, as gcc-3.2 did not have TLS
| support, and gcc-3.3 is only open for regression fixes, so there is
| the question of whether  this patch is OK to install on the branch.
| A strict interpretation of the rules says this is not OK, at least
| not without approval of the branch maintainer, Gaby. 

I'm inclined to accept a simpler patch.

-- Gaby


-- 


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

[Bug target/20286] [3.3 only] gcc panic with __thread attribute

[wilson at gcc dot gnu dot org]

--- Additional Comments From wilson at gcc dot gnu dot org  2005-03-28 
22:40 ---
The patch passed testing, and has been submitted to gcc-patches here:
http://gcc.gnu.org/ml/gcc-patches/2005-03/msg02542.html


-- 


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

[Bug target/20286] [3.3 only] gcc panic with __thread attribute

[cvs-commit at gcc dot gnu dot org]

--- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-03-29 
00:32 ---
Subject: Bug 20286

CVSROOT:/cvs/gcc
Module name:gcc
Branch: gcc-3_3-branch
Changes by: [EMAIL PROTECTED]   2005-03-29 00:32:03

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

Log message:
Fix problem with redefining extern __thread as static __thread.
PR target/20286.
* config/ia64/ia64.c (ia64_encode_section_info): Only abort if encoding
or symbol_str[1] is 's'.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_3-branch&r1=1.16114.2.1061&r2=1.16114.2.1062
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/ia64/ia64.c.diff?cvsroot=gcc&only_with_tag=gcc-3_3-branch&r1=1.198.2.24&r2=1.198.2.25



-- 


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

[Bug target/20286] [3.3 only] gcc panic with __thread attribute

[wilson at gcc dot gnu dot org]

--- Additional Comments From wilson at gcc dot gnu dot org  2005-03-29 
00:33 ---
Fixed.

-- 
   What|Removed |Added

 Status|ASSIGNED|RESOLVED
  Known to work|3.4.0 4.0.0 |3.4.0 4.0.0 3.3.6
 Resolution||FIXED


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

[Bug target/20286] [3.3 only] gcc panic with __thread attribute

[pinskia at gcc dot gnu dot org]

--- Additional Comments From pinskia at gcc dot gnu dot org  2005-03-02 
16:48 ---
Fixed in 3.4.0 by
.

-- 
   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed||1
  Known to fail||3.3.5 3.3.4
  Known to work||3.4.0 4.0.0
   Last reconfirmed|-00-00 00:00:00 |2005-03-02 16:48:16
   date||
Summary|gcc panic with __thread |[3.3 only] gcc panic with
   |attribute   |__thread attribute
   Target Milestone|--- |3.3.6


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