[Bug target/31490] Compile error section type conflict

[Bug target/31490] Compile error section type conflict

[raj dot khem at gmail dot com]

------- Comment #1 from raj dot khem at gmail dot com  2007-04-06 01:28 -------
Created an attachment (id=13333)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13333&action=view)
testcase

testcase to reproduce the problem.


-- 


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

[Bug target/31490] New: Compile error section type conflict

[raj dot khem at gmail dot com]

The attached testcase does not compile for ppc64 using gcc confgured to
generate ppc/ppc64 code.

Attached is a reduced testcase from kernel. This error was not happening with
gcc 4.2 snapshot from January. I think it is related to static inline and
extern inline.

compile the attached testcase 

gcc -m64 -c test.c

test.c:51: error: __ksymtab_cpu_add_sysdev_attr_group causes a section type
conflict


-- 
           Summary: Compile error section type conflict
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: raj dot khem at gmail dot com
 GCC build triplet: i686-pc-linux-gnu
  GCC host triplet: i686-pc-linux-gnu
GCC target triplet: powerpc-*-linux-gnu


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

[Bug target/31490] Compile error section type conflict

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2007-04-06 01:57 -------
Reduced testcase:
int cpu_ (void *attr) {}
const unsigned long _1 __attribute__((section("__"))) =  (unsigned long)&cpu_ ;
const unsigned long _group __attribute__((section("__"))) = 0;


-- 


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

[Bug target/31490] Compile error section type conflict

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2007-04-06 01:59 -------
> This error was not happening with gcc 4.2 snapshot from January. 
My reduced testcase shows it happens even in 4.0.2.  I don't have any thing
older than that right off hand.


-- 


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

[Bug target/31490] Compile error section type conflict

[dtemirbulatov at gmail dot com]

------- Comment #4 from dtemirbulatov at gmail dot com  2007-04-06 03:18 -------
looks like regression is caused by PR26090 fix


-- 


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

[Bug target/31490] Compile error section type conflict

[dtemirbulatov at gmail dot com]

------- Comment #5 from dtemirbulatov at gmail dot com  2007-05-02 16:14 -------
I can reproduce this bug any architecture with -fpic option


-- 


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

[Bug target/31490] Compile error section type conflict

[dtemirbulatov at gmail dot com]

------- Comment #6 from dtemirbulatov at gmail dot com  2007-05-02 16:25 -------
workaround for the bug:
--- gcc/varasm.c-orig   2007-05-02 19:15:04.000000000 +0400
+++ gcc/varasm.c        2007-05-02 19:16:17.000000000 +0400
@@ -5519,6 +5519,8 @@ decl_readonly_section (tree decl, int re
     case SECCAT_RODATA_MERGE_STR_INIT:
     case SECCAT_RODATA_MERGE_CONST:
     case SECCAT_SRODATA:
+    case SECCAT_DATA_REL_RO:
+    case SECCAT_DATA_REL_RO_LOCAL:
       return true;
       break;
     default:


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2007-05-02 16:41 -------
> I can reproduce this bug any architecture with -fpic option

Oh and this is why it fails without -fpic on powerpc64-linux-gnu as really it
is always PIC with the toc based ABI.

>workaround for the bug:
Actually that looks like the correct fix from reading output.h's comment about
these section categories.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|target                      |middle-end
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-05-02 16:41:08
               date|                            |


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

[Bug middle-end/31490] Compile error section type conflict

[segher at kernel dot crashing dot org]

------- Comment #8 from segher at kernel dot crashing dot org  2007-05-15 13:07 -------
Bisecting shows that the original bug (powerpc64 Linux build
errors out) is caused by r122781.  I didn't actually test on 4.2
but the same patch is applied there (as r122782).

If the reduced testcase in comment #2 fails on older compilers,
it is likely a different bug (or an invalid testcase); if a bug, please
file a separate PR for it.


-- 

segher at kernel dot crashing dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |segher at kernel dot
                   |                            |crashing dot org


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

[Bug middle-end/31490] Compile error section type conflict

[dtemirbulatov at gmail dot com]

------- Comment #9 from dtemirbulatov at gmail dot com  2007-05-16 22:47 -------
This patch fixes both testcases

--- gcc/varasm.c.orig   2007-05-16 21:32:11.000000000 +0400
+++ gcc/varasm.c        2007-05-16 21:36:14.000000000 +0400
@@ -5762,12 +5762,14 @@ categorize_decl_for_section (tree decl, 
             be read-only or not, but whether the dynamic link will have to
             do something.  If so, we wish to segregate the data in order to
             minimize cache misses inside the dynamic linker.  */
-         if (reloc & targetm.asm_out.reloc_rw_mask ())
+         if (reloc & targetm.asm_out.reloc_rw_mask ()
+             && !lookup_attribute ("section", DECL_ATTRIBUTES (decl)))
            ret = reloc == 1 ? SECCAT_DATA_REL_LOCAL : SECCAT_DATA_REL;
          else
            ret = SECCAT_DATA;
        }
-      else if (reloc & targetm.asm_out.reloc_rw_mask ())
+      else if (reloc & targetm.asm_out.reloc_rw_mask ()
+              && !lookup_attribute ("section", DECL_ATTRIBUTES (decl)))
        ret = reloc == 1 ? SECCAT_DATA_REL_RO_LOCAL : SECCAT_DATA_REL_RO;
       else if (reloc || flag_merge_constants < 2)
        /* C and C++ don't allow different variables to share the same

, bootstraped and tested on i686-pc-linux-gnu with no new regressions, tested
4.2 version with gdb-sim on following architectures arm-elf, arm-none-eabi,
mips-elf, mipsisa64-elf, powerpc-eabisim, sh-elf, powerpc-unknown-eabispe with
no new regressions


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[segher at kernel dot crashing dot org]

------- Comment #10 from segher at kernel dot crashing dot org  2007-05-18 14:57 -------
Created an attachment (id=13578)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13578&action=view)
proposed patch

still need to run the testsuite on it


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[schwab at suse dot de]

------- Comment #11 from schwab at suse dot de  2007-06-20 11:32 -------
Also broken on ia64.


-- 

schwab at suse dot de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 GCC target triplet|powerpc64-*-linux-gnu       |powerpc64-*-linux-gnu,ia64-
                   |                            |*-*


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

[Bug middle-end/31490] Compile error section type conflict

[janis at gcc dot gnu dot org]

------- Comment #12 from janis at gcc dot gnu dot org  2007-08-27 21:54 -------
Trunk currently fails on powerpc64-linux building
libstdc++-v3/src/system_error.cc.  Hacking libtool a bit allows seeing the
message "std::system_category causes a section type conflict".  The file is
compiled with -O2 and -fdata-sections; if either of those is removed then the
bootstrap succeeds.

This PR looked related (and Andrew Pinski confirmed that it is) so I tried the
patch from comment #10; it needs more parens to avoid warnings, but with that
patch the bootstrap succeeds.


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[dtemirbulatov at gmail dot com]

------- Comment #13 from dtemirbulatov at gmail dot com  2007-08-28 16:25 -------
reviewed patch posted here
http://gcc.gnu.org/ml/gcc-patches/2007-05/msg01280.html


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[janis at gcc dot gnu dot org]

------- Comment #14 from janis at gcc dot gnu dot org  2007-08-28 18:15 -------
The patch that I tried is the one attached for comment #10, not the one in
comment #9.  With that patch a bootstrap of all languages but Ada succeeded. 
The test results look reasonable, although I don't have results from the last
few days with which to compare them.


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[drow at gcc dot gnu dot org]

------- Comment #15 from drow at gcc dot gnu dot org  2007-08-31 17:42 -------
Has Segher's patch been sent to gcc-patches?  I've just encountered the ppc64
bootstrap failure too.


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[etiennes at cse dot unsw dot edu dot au]

------- Comment #16 from etiennes at cse dot unsw dot edu dot au  2007-11-28 05:11 -------
I just tried compiling 2.6.23.9 ia64 and the compile failed citing
drivers/char/ipmi/ipmi_si_intf.c:1095: error: __param_hotmod causes a section
type conflict

gcc (GCC) 4.2.3 20071123 (prerelease) (Debian 4.2.2-4)


-- 


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

[Bug middle-end/31490] Compile error section type conflict

[sje at cup dot hp dot com]

------- Comment #17 from sje at cup dot hp dot com  2010-04-16 22:29 -------
Is there any reason none of the patches created for this bug have been checked
in?  I still get a 'section type conflict' on IA64 with the test case from
Comment #2.


-- 

sje at cup dot hp dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sje at cup dot hp dot com


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

[Bug middle-end/31490] Compile error section type conflict

[sje at gcc dot gnu.org]

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

--- Comment #20 from Steve Ellcey <sje at gcc dot gnu.org> 2010-11-17 22:55:05 UTC ---
Author: sje
Date: Wed Nov 17 22:55:01 2010
New Revision: 166888

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166888
Log:
2010-11-17  Steve Ellcey  <sje@cup.hp.com>

    PR middle-end/31490
    * gcc.dg/pr31490.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/pr31490.c
Modified:
    trunk/gcc/testsuite/ChangeLog

[Bug middle-end/31490] Compile error section type conflict

[sje at gcc dot gnu.org]

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

--- Comment #19 from Steve Ellcey <sje at gcc dot gnu.org> 2010-11-17 22:54:07 UTC ---
Author: sje
Date: Wed Nov 17 22:54:05 2010
New Revision: 166887

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166887
Log:
2010-11-17  Dinar Temirbulatov <dtemirbulatov@gmail.com>
        Steve Ellcey  <sje@cup.hp.com>

    PR middle-end/31490
    * varasm.c (categorize_decl_for_section): Ignore reloc_rw_mask
    if section attribute used.

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

[Bug middle-end/31490] Compile error section type conflict

[nicolas.veyssiere at thalesgroup dot com]

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

Nicolas VEYSSIERE <nicolas.veyssiere at thalesgroup dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nicolas.veyssiere at
                   |                            |thalesgroup dot com

--- Comment #18 from Nicolas VEYSSIERE <nicolas.veyssiere at thalesgroup dot com> 2010-11-09 10:58:14 UTC ---
Hello, I have the same question as #17, I am using GCC on ia64 too, and this
problem is still in the last version of GCC (4.4.5), and if I apply the patch
of Dinar Temirbulatov (#9) it seems to works, (maybe I have to use the patch
proposed by Segher Boessenkool, #10, i don't know).