[Bug driver/52297] New: FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[Bug driver/52297] New: FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[danglin at gcc dot gnu.org]

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

             Bug #: 52297
           Summary: FAIL: gcc.dg/lto/trans-mem-1
                    c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link,
                    -flto -fgnu-tm
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: driver
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: danglin@gcc.gnu.org
              Host: hppa64-hp-hpux11.11
            Target: hppa64-hp-hpux11.11
             Build: hppa64-hp-hpux11.11


Executing on host: /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/  
-
flto -fgnu-tm  -c  -o c_lto_trans-mem-1_0.o
/test/gnu/gcc/gcc/gcc/testsuite/gcc.
dg/lto/trans-mem-1_0.c    (timeout = 300)
spawn /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/ -flto -fgnu-tm
-
c -o c_lto_trans-mem-1_0.o
/test/gnu/gcc/gcc/gcc/testsuite/gcc.dg/lto/trans-mem-
1_0.c
PASS: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o assemble, -flto -fgnu-tm
Executing on host: /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/  
-
flto -fgnu-tm  -c  -o c_lto_trans-mem-1_1.o
/test/gnu/gcc/gcc/gcc/testsuite/gcc.
dg/lto/trans-mem-1_1.c    (timeout = 300)
spawn /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/ -flto -fgnu-tm
-
c -o c_lto_trans-mem-1_1.o
/test/gnu/gcc/gcc/gcc/testsuite/gcc.dg/lto/trans-mem-1_1.c
PASS: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_1.o assemble, -flto -fgnu-tm
Executing on host: /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/
c_lto_trans-mem-1_0.o c_lto_trans-mem-1_1.o   -flto -fgnu-tm       -o
gcc-dg-lto-trans-mem-1-01.exe    (timeout = 300)
spawn /test/gnu/gcc/objdir/gcc/xgcc -B/test/gnu/gcc/objdir/gcc/
c_lto_trans-mem-1_0.o c_lto_trans-mem-1_1.o -flto -fgnu-tm -o
gcc-dg-lto-trans-mem-1-01.exe
xgcc: error: libitm.spec: No such file or directory
compiler exited with status 1
output is:
xgcc: error: libitm.spec: No such file or directory

FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link,
-flto -fgnu-tm
UNRESOLVED: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o
execute -flto -fgnu-tm

Similar fails:

FAIL: gcc.dg/lto/trans-mem-2 c_lto_trans-mem-2_0.o-c_lto_trans-mem-2_1.o link,
-flto -fgnu-tm
FAIL: gcc.dg/lto/trans-mem-4 c_lto_trans-mem-4_0.o-c_lto_trans-mem-4_1.o link,
-flto -fgnu-tm

-bash-3.2$ ./xgcc -B./ -v
Reading specs from ./specs
COLLECT_GCC=./xgcc
COLLECT_LTO_WRAPPER=./lto-wrapper
Target: hppa64-hp-hpux11.11
Configured with: ../gcc/configure --with-gnu-as --with-as=/opt/gnu64/bin/as
--with-ld=/usr/ccs/bin/ld --enable-shared --with-local-prefix=/opt/gnu64
--prefix=/opt/gnu64/gcc/gcc-4.7 --build=hppa64-hp-hpux11.11
--enable-threads=posix --disable-nls --with-gmp=/opt/gnu64/gcc/gmp
--with-libelf=/opt/gnu64 --enable-languages=c,c++,objc,obj-c++,fortran
Thread model: posix
gcc version 4.7.0 20120216 (experimental) [trunk revision 184311] (GCC)

[Bug driver/52297] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-02-17
                 CC|                            |ebotcazou at gcc dot
                   |                            |gnu.org
     Ever Confirmed|0                           |1

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-02-17 15:45:29 UTC ---
AFAICT this affects also most of (all?) the *-linux-* platforms,e.g.,
http://glutton.geoffk.org/HEAD/native-logsum/gcc/testsuite/gcc/gcc.log.gzip.
The failures appeared between revisions 184169 and 184177 (see
http://gcc.gnu.org/ml/gcc-regression/2012-02/msg00077.html and
http://gcc.gnu.org/ml/gcc-regression/2012-02/msg00071.html), likely revision
184174.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|hppa64-hp-hpux11.11         |hppa64-hp-hpux11.11,
                   |                            |x86_64-*-*
          Component|driver                      |testsuite
   Target Milestone|---                         |4.7.0
            Summary|FAIL:                       |[4.7 Regression] FAIL:
                   |gcc.dg/lto/trans-mem-1      |gcc.dg/lto/trans-mem-1
                   |c_lto_trans-mem-1_0.o-c_lto |c_lto_trans-mem-1_0.o-c_lto
                   |_trans-mem-1_1.o link,      |_trans-mem-1_1.o link,
                   |-flto -fgnu-tm              |-flto -fgnu-tm

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-02-20 12:15:08 UTC ---
Yes, I'm seeing this as well.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[dominiq at lps dot ens.fr]

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

--- Comment #3 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-02-20 13:22:36 UTC ---
The tests pass on powerpc-apple-darwin9 and x86_64-apple-darwin10, but not on
i686-pc-linux-gnu; so x86_64-*-* does not seem the right target descriptor.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[ebotcazou at gcc dot gnu.org]

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

--- Comment #4 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-02-21 10:13:57 UTC ---
That's (essentially) independent of the target, as it's a pure testsuite issue.
Tests that require linking with -litm should go in libitm/testsuite instead.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
                 CC|                            |aldyh at gcc dot gnu.org,
                   |                            |jakub at gcc dot gnu.org,
                   |                            |torvald at gcc dot gnu.org

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[aldyh at gcc dot gnu.org]

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

--- Comment #5 from Aldy Hernandez <aldyh at gcc dot gnu.org> 2012-02-22 20:00:31 UTC ---
(In reply to comment #4)
> That's (essentially) independent of the target, as it's a pure testsuite issue.
> Tests that require linking with -litm should go in libitm/testsuite instead.

It shouldn't be linking with -litm.  All the required dummy functions should be
in gcc.dg/lto/trans-mem.h.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[jakub at gcc dot gnu.org]

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

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-02-22 20:08:46 UTC ---
(In reply to comment #5)
> (In reply to comment #4)
> > That's (essentially) independent of the target, as it's a pure testsuite issue.
> > Tests that require linking with -litm should go in libitm/testsuite instead.
> 
> It shouldn't be linking with -litm.  All the required dummy functions should be
> in gcc.dg/lto/trans-mem.h.

But -fgnu-tm during linking now results in libitm.spec being sourced in, so it
needs either a hack version thereof somewhere where it can find it through -B
options, or some other approach.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[aldyh at redhat dot com]

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

--- Comment #7 from Aldy Hernandez <aldyh at redhat dot com> 2012-02-22 20:20:34 UTC ---
On 02/22/12 14:08, jakub at gcc dot gnu.org wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52297
>
> --- Comment #6 from Jakub Jelinek<jakub at gcc dot gnu.org>  2012-02-22 20:08:46 UTC ---
> (In reply to comment #5)
>> (In reply to comment #4)
>>> That's (essentially) independent of the target, as it's a pure testsuite issue.
>>> Tests that require linking with -litm should go in libitm/testsuite instead.
>>
>> It shouldn't be linking with -litm.  All the required dummy functions should be
>> in gcc.dg/lto/trans-mem.h.
>
> But -fgnu-tm during linking now results in libitm.spec being sourced in, so it
> needs either a hack version thereof somewhere where it can find it through -B
> options, or some other approach.
>

This looks like a regression brought about by:

+2012-02-13  Eric Botcazou  <ebotcazou@adacore.com>
+
+       * gcc.c (LINK_COMMAND_SPEC): Deal with -fgnu-tm.
+       (GTM_SELF_SPECS): Define if not already defined.
+       (driver_self_specs): Add GTM_SELF_SPECS.
+        * config/darwin.h (LINK_COMMAND_SPEC_A): Deal with -fgnu-tm.
+        (GTM_SELF_SPECS): Define.
+       * config/i386/cygwin.h (GTM_SELF_SPECS): Likewise.
+       * config/i386/mingw32.h (GTM_SELF_SPECS): Likewise.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[ebotcazou at gcc dot gnu.org]

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

--- Comment #8 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-02-23 17:54:26 UTC ---
> This looks like a regression brought about by:
> 
> +2012-02-13  Eric Botcazou  <ebotcazou@adacore.com>
> +
> +       * gcc.c (LINK_COMMAND_SPEC): Deal with -fgnu-tm.
> +       (GTM_SELF_SPECS): Define if not already defined.
> +       (driver_self_specs): Add GTM_SELF_SPECS.
> +        * config/darwin.h (LINK_COMMAND_SPEC_A): Deal with -fgnu-tm.
> +        (GTM_SELF_SPECS): Define.
> +       * config/i386/cygwin.h (GTM_SELF_SPECS): Likewise.
> +       * config/i386/mingw32.h (GTM_SELF_SPECS): Likewise.

Well, the existing linking scheme was totally broken, which allowed you to
devise the gcc.dg/lto/trans-mem.h hack.  Now that it has been fixed, the hack
doesn't work anymore, which isn't very surprising...  We need a proper fix
instead.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[rguenth at gcc dot gnu.org]

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

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

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

--- Comment #10 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-02-29 10:06:16 UTC ---
Fixed.

[Bug testsuite/52297] [4.7 Regression] FAIL: gcc.dg/lto/trans-mem-1 c_lto_trans-mem-1_0.o-c_lto_trans-mem-1_1.o link, -flto -fgnu-tm

[rguenth at gcc dot gnu.org]

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

--- Comment #9 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-02-29 10:06:00 UTC ---
Author: rguenth
Date: Wed Feb 29 10:05:55 2012
New Revision: 184657

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=184657
Log:
2012-02-29  Richard Guenther  <rguenther@suse.de>

    PR testsuite/52297
    * gcc.dg/lto/trans-mem-1_0.c: Remove.
    * gcc.dg/lto/trans-mem-1_1.c: Likewise.
    * gcc.dg/lto/trans-mem-2_0.c: Likewise.
    * gcc.dg/lto/trans-mem-2_1.c: Likewise.
    * gcc.dg/lto/trans-mem-4_0.c: Likewise.
    * gcc.dg/lto/trans-mem-4_1.c: Likewise.

Removed:
    trunk/gcc/testsuite/gcc.dg/lto/trans-mem-1_0.c
    trunk/gcc/testsuite/gcc.dg/lto/trans-mem-1_1.c
    trunk/gcc/testsuite/gcc.dg/lto/trans-mem-2_0.c
    trunk/gcc/testsuite/gcc.dg/lto/trans-mem-2_1.c
    trunk/gcc/testsuite/gcc.dg/lto/trans-mem-4_0.c
    trunk/gcc/testsuite/gcc.dg/lto/trans-mem-4_1.c
Modified:
    trunk/gcc/testsuite/ChangeLog