[Bug testsuite/55994] multiple definition or memset or strlen for builtins tests with LTO options

2013-03-22 Thread joel at gcc dot gnu.org


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



Joel Sherrill joel at gcc dot gnu.org changed:



   What|Removed |Added



 Status|NEW |RESOLVED

 CC||joel at gcc dot gnu.org

 Resolution||FIXED



--- Comment #8 from Joel Sherrill joel at gcc dot gnu.org 2013-03-22 16:03:14 
UTC ---

Sebastian's patch applied to the head and 4.8 branch. Closing.


[Bug testsuite/55994] multiple definition or memset or strlen for builtins tests with LTO options

2013-03-20 Thread sebastian.hu...@embedded-brains.de


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



Sebastian Huber sebastian.hu...@embedded-brains.de changed:



   What|Removed |Added



 CC||sebastian.huber@embedded-br

   ||ains.de



--- Comment #7 from Sebastian Huber sebastian.hu...@embedded-brains.de 
2013-03-20 09:20:24 UTC ---

The crucial thing is also that all these functions are recognized as builtins

and GCC knows what they are expected to do.  The failure also vanishes if

-fno-builtin is used.



Lets have a look at this example:



gcc 1.c 2.c 3.c -l4



I expect that the modules 1.o 2.o 3.o end up in the executable and unresolved

symbols inside this module set are resolved with definitions provided by the

library 4.  In particular if I have a global function f() in 2.c then I expect

to have this function in my executable even in case a global function f() is

also defined in the library 4.



This is wrong for builtins.  Here GCC seems to assume that implementations are

interchangeable.  It would be quite interesting to know what happens if we

compile libc with -flto.



In one failed GCC test f() is memcpy(), 2.c is 20010124-1-lib.c and lib4 is

libc.


[Bug testsuite/55994] multiple definition or memset or strlen for builtins tests with LTO options

2013-01-24 Thread janis at gcc dot gnu.org


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



--- Comment #6 from Janis Johnson janis at gcc dot gnu.org 2013-01-25 
00:26:43 UTC ---

Author: janis

Date: Fri Jan 25 00:26:34 2013

New Revision: 195458



URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=195458

Log:

Backport from mainline

2013-01-16  Janis Johnson  jani...@codesourcery.com



PR testsuite/55994

* gcc.c-torture/execute/builtins/builtins.exp: Add

-Wl,--allow-multiple-definition for eabi and elf targets.



Modified:

branches/gcc-4_7-branch/gcc/testsuite/ChangeLog

   

branches/gcc-4_7-branch/gcc/testsuite/gcc.c-torture/execute/builtins/builtins.exp


[Bug testsuite/55994] multiple definition or memset or strlen for builtins tests with LTO options

2013-01-16 Thread janis at gcc dot gnu.org


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



--- Comment #5 from Janis Johnson janis at gcc dot gnu.org 2013-01-16 
18:52:56 UTC ---

Author: janis

Date: Wed Jan 16 18:52:51 2013

New Revision: 195250



URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=195250

Log:

PR testsuite/55994

* gcc.c-torture/execute/builtins/builtins.exp: Add

-Wl,--allow-multiple-definition for eabi and elf targets.



Modified:

trunk/gcc/testsuite/ChangeLog

trunk/gcc/testsuite/gcc.c-torture/execute/builtins/builtins.exp


[Bug testsuite/55994] multiple definition or memset or strlen for builtins tests with LTO options

2013-01-15 Thread janis at gcc dot gnu.org


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



--- Comment #4 from Janis Johnson janis at gcc dot gnu.org 2013-01-15 
20:00:17 UTC ---

Patch posted: http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00801.html.