[Bug middle-end/6123] sparc-sun-solaris2.7 gcc-3.1 C testsuite failure in verify_local_live_at_start w/-fpic|-fPIC on gcc.dg/20020312-2.c

[Bug middle-end/6123] sparc-sun-solaris2.7 gcc-3.1 C testsuite failure in verify_local_live_at_start w/-fpic|-fPIC on gcc.dg/20020312-2.c

[dhazeghi@yahoo.com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


dhazeghi@yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  GCC build triplet|                            |sparc-sun-solaris2.7
   GCC host triplet|                            |sparc-sun-solaris2.7
 GCC target triplet|                            |sparc-sun-solaris2.7


------- Additional Comments From dhazeghi@yahoo.com  2003-06-05 03:37 -------
Confirmed with gcc 3.2. With gcc 3.3 (branch) on i686-linux and Christian's testcase, I get:

/tmp/ccS0QCPS.s: Assembler messages:
/tmp/ccS0QCPS.s:20: Error: suffix or operands invalid for `lea'

With mainline, the testcase compiles and assembles fine.

On cross-compiler to SPARC, Christian's testcase compiles fine with gcc 3.3 branch and mainline 
(20030604).

Does this mean this bug has been fixed? Thanks,

Dara





------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug middle-end/6123] sparc-sun-solaris2.7 gcc-3.1 C testsuite failure in verify_local_live_at_start w/-fpic|-fPIC on gcc.dg/20020312-2.c

[ghazi@gcc.gnu.org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From ghazi@gcc.gnu.org  2003-06-09 05:57 -------
The problem persists.
http://gcc.gnu.org/ml/gcc-testresults/2003-06/msg00487.html

[Bug middle-end/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2004-07-10 20:00 -------
Corrected summary to advertise a more accurate description.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
  GCC build triplet|sparc-sun-solaris2.7        |sparc-sun-solaris2.*
   GCC host triplet|sparc-sun-solaris2.7        |sparc-sun-solaris2.*
 GCC target triplet|sparc-sun-solaris2.7        |sparc-sun-solaris2.*
           Keywords|ice-on-valid-code           |
            Summary|gcc-3.1 C testsuite failure |__pic__/__PIC__ not defined
                   |in                          |when -fpic/-fPIC is
                   |verify_local_live_at_start  |specified
                   |w/-fpic|-fPIC on            |
                   |gcc.dg/20020312-2.c         |


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-10-09 00:31 -------
This is a target problem.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|middle-end                  |target


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[zack at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
  BugsThisDependsOn|                            |18382


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[zack at gcc dot gnu dot org]

------- Additional Comments From zack at gcc dot gnu dot org  2004-11-08 20:37 -------
I've filed bug 18382 for the general solution to this problem.

-- 


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[kghazi at verizon dot net]

------- Additional Comments From kghazi at verizon dot net  2004-11-08 20:45 -------
Subject: Re:  __pic__/__PIC__ not defined when -fpic/-fPIC is specified

I submitted a patch to fix this back in April:

http://gcc.gnu.org/ml/gcc-patches/2004-04.argh/msg00168.html



-- 


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[zack at codesourcery dot com]

------- Additional Comments From zack at codesourcery dot com  2004-11-08 21:23 -------
Subject: Re:  __pic__/__PIC__ not defined when -fpic/-fPIC is specified


Ah.  This is why I thought it had already been done.  Maybe you want
to poke Alan Modra about the research he said he would do?

zw


-- 


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[amodra at bigpond dot net dot au]

------- Additional Comments From amodra at bigpond dot net dot au  2004-11-10 05:07 -------
I made the following reply the day after I said I'd poke around.  Somehow it
seems to be missing from the gcc-patches archive.  Anyway, here it is again:


I grepped through the entire slackware linux source base for occurrences
of __PIC__, and found that all those that might affect PowerPC64 were
testing for (b).  There were only three occurrences, all in glibc!  Many
more for x86 which I didn't pay much attention to, but the few I did
look at were actually testing for (a).  This isn't so surprising, since
on x86 we want different assembly for PIC code, regardless of whether
PIC code is for shared libs (most likely) or otherwise.  OTOH, the
only sensible test of __PIC__ on PowerPC64 can be for (b), since we
are (a) all the time.

Because we currently define __PIC__ all the time, the three occurrences
affecting PowerPC64 Linux result in missed optimizations for non-shared
lib code.

This suggests that powerpc64-linux-gcc should only define __PIC__ when
-fPIC is given, and that Kaveh's patch is OK.


-- 


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[amodra at bigpond dot net dot au]

------- Additional Comments From amodra at bigpond dot net dot au  2004-11-10 05:13 -------
No, actually not the next day, but a few days later.  Incidentally, the (a) and
(b) are from an earlier reply of mine on this subject:

On most other targets, -fpic, ie. flag_pic, means two things:
a) generate PIC code, and
b) generate code suitable for ELF shared libraries.
I'll not argue the wisdom of -fpic meaning two things, but that's just
how things evolved.  On PowerPC64, code generation is always PIC, but we
don't want the overhead of meeting shared library run-time symbol
resolution requirements when generating application code.  Thus, we
don't want to set flag_pic all the time.



-- 


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-11-16 22:42 -------
Subject: Bug 6123

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	jsm28@gcc.gnu.org	2004-11-16 22:42:21

Modified files:
	gcc            : ChangeLog 
	gcc/config     : sol2.h 

Log message:
	2004-11-16  Daniel Jacobowitz  <dan@codesourcery.com>
	Mark Mitchell  <mark@codesourcery.com>
	
	PR target/6123
	* config/sol2.h (TARGET_OS_CPP_BUILTINS): Define __PIC__ and
	__pic__ if PIC.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.6367&r2=2.6368
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/config/sol2.h.diff?cvsroot=gcc&r1=1.15&r2=1.16



-- 


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

[Bug target/6123] __pic__/__PIC__ not defined when -fpic/-fPIC is specified

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2005-01-03 21:03 -------
This will be fixed on Solaris in the upcoming 4.0.0.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.0.0


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