public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
@ 2020-03-23 10:14 marxin at gcc dot gnu.org
  2020-03-23 10:15 ` [Bug ipa/94271] " marxin at gcc dot gnu.org
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-23 10:14 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

            Bug ID: 94271
           Summary: [8/9/10 Regression] lto1: error: two or more sections
                    for .gnu.lto_fast_clamp.default.9.564bf999b130b5e
                    since r8-1461-g871cc215f7507cbe
           Product: gcc
           Version: 10.0
            Status: UNCONFIRMED
          Keywords: rejects-valid
          Severity: normal
          Priority: P3
         Component: ipa
          Assignee: unassigned at gcc dot gnu.org
          Reporter: marxin at gcc dot gnu.org
                CC: marxin at gcc dot gnu.org
  Target Milestone: ---

Since the revision we do not accept:

$ cat 1.i
int a;
static inline int
__attribute__((target_clones("default", "sse2", "sse3", "sse4.1", "sse4.2",
                             "popcnt", "avx", "avx2", "avx512f",
                             "fma4"))) fast_clamp() {}
void c() { a = fast_clamp(); }

$ cat 2.i
int aa;
static inline int
__attribute__((target_clones("default", "sse2", "sse3", "sse4.1", "sse4.2",
                             "popcnt", "avx", "avx2", "avx512f",
                             "fma4"))) fast_clamp() {}
void b() { aa = fast_clamp(); }

int main()
{
  return 0;
}

$ gcc 1.i 2.i  -O2 -flto
...
(.text+0x0): multiple definition of `fast_clamp'; /tmp/cc3Vv3xl.o (symbol from
plugin):(.text+0x0): first defined here

The situation is even worse since
r10-6242-gc2bd2b4664be8b73f8fd58a64dec1e93871797cc:

/usr/bin/ld: /tmp/ccwTYmhr.o (symbol from plugin): in function `main':
(.text+0x0): multiple definition of `fast_clamp'; /tmp/ccdQcwap.o (symbol from
plugin):(.text+0x0): first defined here
1.i:6:1: error: ‘fast_clamp’ has already been defined
    6 | void c() { a = fast_clamp(); }
      | ^
1.i:6:1: note: previously defined here
lto1: fatal error: errors during merging of translation units
compilation terminated.
lto-wrapper: fatal error: gcc returned 1 exit status
compilation terminated.
/usr/bin/ld: error: lto-wrapper failed
collect2: error: ld returned 1 exit status

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
@ 2020-03-23 10:15 ` marxin at gcc dot gnu.org
  2020-03-23 13:23 ` marxin at gcc dot gnu.org
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-23 10:15 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2020-03-23
           Assignee|unassigned at gcc dot gnu.org      |marxin at gcc dot gnu.org

--- Comment #1 from Martin Liška <marxin at gcc dot gnu.org> ---
I'm working on that..

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
  2020-03-23 10:15 ` [Bug ipa/94271] " marxin at gcc dot gnu.org
@ 2020-03-23 13:23 ` marxin at gcc dot gnu.org
  2020-03-23 14:10 ` rguenth at gcc dot gnu.org
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-23 13:23 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |10.0, 8.4.0, 9.3.0
      Known to work|                            |7.4.0

--- Comment #2 from Martin Liška <marxin at gcc dot gnu.org> ---
Without LTO it fails to build with r10-6242-gc2bd2b4664be8b73.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
  2020-03-23 10:15 ` [Bug ipa/94271] " marxin at gcc dot gnu.org
  2020-03-23 13:23 ` marxin at gcc dot gnu.org
@ 2020-03-23 14:10 ` rguenth at gcc dot gnu.org
  2020-03-25 10:04 ` cvs-commit at gcc dot gnu.org
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: rguenth at gcc dot gnu.org @ 2020-03-23 14:10 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |8.5
           Priority|P3                          |P2

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2020-03-23 14:10 ` rguenth at gcc dot gnu.org
@ 2020-03-25 10:04 ` cvs-commit at gcc dot gnu.org
  2020-03-25 10:04 ` [Bug ipa/94271] [8/9 " marxin at gcc dot gnu.org
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-03-25 10:04 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:

https://gcc.gnu.org/g:724ec02c2c6d1b79788be77f68ebb6ca7b5b6acd

commit r10-7372-g724ec02c2c6d1b79788be77f68ebb6ca7b5b6acd
Author: Martin Liska <mliska@suse.cz>
Date:   Wed Mar 25 11:03:39 2020 +0100

    Make target_clones resolver fn static if possible.

            PR target/93274
            PR ipa/94271
            * config/i386/i386-features.c (make_resolver_func): Drop
            public flag for resolver.
            * config/rs6000/rs6000.c (make_resolver_func): Add comdat
            group for resolver and drop public flag if possible.
            * multiple_target.c (create_dispatcher_calls): Drop unique_name
            and resolution as we want to enable LTO privatization of the
default
            symbol.
            PR target/93274
            PR ipa/94271
            * gcc.target/i386/pr81213-2.c: New test.
            * gcc.target/i386/pr81213.c: Add additional source.
            * gcc.dg/lto/pr94271_0.c: New test.
            * gcc.dg/lto/pr94271_1.c: New test.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2020-03-25 10:04 ` cvs-commit at gcc dot gnu.org
@ 2020-03-25 10:04 ` marxin at gcc dot gnu.org
  2020-03-26  9:08 ` ro at gcc dot gnu.org
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-25 10:04 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[8/9/10 Regression] lto1:   |[8/9 Regression] lto1:
                   |error: two or more sections |error: two or more sections
                   |for                         |for
                   |.gnu.lto_fast_clamp.default |.gnu.lto_fast_clamp.default
                   |.9.564bf999b130b5e since    |.9.564bf999b130b5e since
                   |r8-1461-g871cc215f7507cbe   |r8-1461-g871cc215f7507cbe
      Known to work|                            |10.0
      Known to fail|10.0                        |

--- Comment #4 from Martin Liška <marxin at gcc dot gnu.org> ---
Fixed on master.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2020-03-25 10:04 ` [Bug ipa/94271] [8/9 " marxin at gcc dot gnu.org
@ 2020-03-26  9:08 ` ro at gcc dot gnu.org
  2020-03-26  9:26 ` marxin at gcc dot gnu.org
  2020-03-31 10:48 ` marxin at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: ro at gcc dot gnu.org @ 2020-03-26  9:08 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

Rainer Orth <ro at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ro at gcc dot gnu.org

--- Comment #5 from Rainer Orth <ro at gcc dot gnu.org> ---
Unfortunately, apart from PR testsuite/94334, the patch introduced another
failure:

+FAIL: gcc.target/i386/pr81213-2.c (test for excess errors)

Excess errors:
/vol/gcc/src/hg/master/local/gcc/testsuite/gcc.target/i386/pr81213-2.c:3:1:
error: the call requires 'ifunc', which is not supported by this target

on i386-pc-solaris2.11 (and probably others).

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2020-03-26  9:08 ` ro at gcc dot gnu.org
@ 2020-03-26  9:26 ` marxin at gcc dot gnu.org
  2020-03-31 10:48 ` marxin at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-26  9:26 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

--- Comment #6 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to Rainer Orth from comment #5)
> Unfortunately, apart from PR testsuite/94334, the patch introduced another
> failure:
> 
> +FAIL: gcc.target/i386/pr81213-2.c (test for excess errors)
> 
> Excess errors:
> /vol/gcc/src/hg/master/local/gcc/testsuite/gcc.target/i386/pr81213-2.c:3:1:
> error: the call requires 'ifunc', which is not supported by this target
> 
> on i386-pc-solaris2.11 (and probably others).

Fixed with g:d6730f06420106af01e26414f046626b5292565d.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [Bug ipa/94271] [8/9 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe
  2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
                   ` (6 preceding siblings ...)
  2020-03-26  9:26 ` marxin at gcc dot gnu.org
@ 2020-03-31 10:48 ` marxin at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-03-31 10:48 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94271

Martin Liška <marxin at gcc dot gnu.org> changed:

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

--- Comment #7 from Martin Liška <marxin at gcc dot gnu.org> ---
I tent to close it as I'm not planning any backports, target_clones is quite a
fragile feature.

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-03-31 10:48 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-23 10:14 [Bug ipa/94271] New: [8/9/10 Regression] lto1: error: two or more sections for .gnu.lto_fast_clamp.default.9.564bf999b130b5e since r8-1461-g871cc215f7507cbe marxin at gcc dot gnu.org
2020-03-23 10:15 ` [Bug ipa/94271] " marxin at gcc dot gnu.org
2020-03-23 13:23 ` marxin at gcc dot gnu.org
2020-03-23 14:10 ` rguenth at gcc dot gnu.org
2020-03-25 10:04 ` cvs-commit at gcc dot gnu.org
2020-03-25 10:04 ` [Bug ipa/94271] [8/9 " marxin at gcc dot gnu.org
2020-03-26  9:08 ` ro at gcc dot gnu.org
2020-03-26  9:26 ` marxin at gcc dot gnu.org
2020-03-31 10:48 ` marxin at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).