[Bug c/18382] New: define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[Bug c/18382] New: define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[zack at gcc dot gnu dot org]

The __pic__ and __PIC__ macros are currently defined scattershot in target
configuration files, some using specs %{pic:-D...} notation and some using
builtin_define() from TARGET_*_CPP_BUILTINS.  It would be better to define
them in a consistent fashion (__pic__ only if -fpic, both if -fPIC) in
c-cppbuiltin.c.  See bug 6123 for examples of what this would help.

I'm tasking myself with this, but I'm not going to do it until 4.1.

-- 
           Summary: define __pic__ and/or __PIC__ in c-cppbuiltins.c instead
                    of scattershot in target config
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P2
         Component: c
        AssignedTo: zack at gcc dot gnu dot org
        ReportedBy: zack at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org
GCC target triplet: *-*-*
OtherBugsDependingO 6123
             nThis:


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

[Bug c/18382] define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[zack at gcc dot gnu dot org]

------- Additional Comments From zack at gcc dot gnu dot org  2004-11-08 20:36 -------
accept, set milestone.

Just to be clear, I'm not doing this until 4.1 because global changes to target
configury would be too destabilizing right now.  We can live with scattershot
for one more release.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2004-11-08 20:36:12
               date|                            |
   Target Milestone|---                         |4.1.0


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

[Bug c/18382] define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[ghazi at gcc dot gnu dot org]

------- Comment #5 from ghazi at gcc dot gnu dot org  2005-11-22 03:27 -------
Updated patch installed on mainline:
http://gcc.gnu.org/ml/gcc-patches/2005-11/msg01575.html


-- 

ghazi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://gcc.gnu.org/ml/gcc-  |http://gcc.gnu.org/ml/gcc-
                   |patches/2005-               |patches/2005-
                   |09/msg00302.html            |11/msg01575.html
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.2.0


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

[Bug c/18382] define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[ghazi at gcc dot gnu dot org]

------- Additional Comments From ghazi at gcc dot gnu dot org  2005-09-07 14:18 -------
Updated patch for mainline here:
http://gcc.gnu.org/ml/gcc-patches/2005-09/msg00302.html

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |ghazi at gcc dot gnu dot org
                   |dot org                     |
                URL|http://gcc.gnu.org/ml/gcc-  |http://gcc.gnu.org/ml/gcc-
                   |patches/2004-               |patches/2005-
                   |04/msg00168.html            |09/msg00302.html
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2005-08-24 19:06:19         |2005-09-07 14:18:13
               date|                            |


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

[Bug c/18382] define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[ghazi at gcc dot gnu dot org]

------- Additional Comments From ghazi at gcc dot gnu dot org  2005-08-24 19:06 -------
By the way, a patch to fix this in early 2004 was posted here:
http://gcc.gnu.org/ml/gcc-patches/2004-04/msg00168.html

I never got an official "okay" in the discussion thread:
http://gcc.gnu.org/ml/gcc-patches/2004-04/msg00507.html



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2004-
                   |                            |04/msg00168.html
   Last reconfirmed|2005-02-07 03:45:45         |2005-08-24 19:06:19
               date|                            |


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

[Bug c/18382] define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-07-05 02:04 -------
Since Zack is no longer around, changing the owner back to the pool.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|zack at codesourcery dot com|unassigned at gcc dot gnu
                   |                            |dot org
             Status|ASSIGNED                    |NEW


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

[Bug c/18382] define __pic__ and/or __PIC__ in c-cppbuiltins.c instead of scattershot in target config

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.0                       |---


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