From: Jakub Jelinek <jakub@redhat.com>
To: Richard Biener <rguenther@suse.de>, Jan Hubicka <jh@suse.cz>
Cc: gcc-patches@gcc.gnu.org
Subject: [PATCH] Make LTO tests that require recent binutils UNSUPPORTED with older ones (PR lto/86004)
Date: Tue, 11 Dec 2018 07:31:00 -0000 [thread overview]
Message-ID: <20181211073122.GL12380@tucnak> (raw)
Hi!
As mentioned in the PR, older binutils (like 2.25) complain on these tests
when using -r that:
plugin needed to handle lto object
The following patch introduces an effective target and guards those tests on
a linker with this issue fixed.
Regtested on x86_64-linux and i686-linux, plus regtested on x86_64-linux
with 2.25 binutils, where the patch converts all those FAILs into
UNSUPPORTEDs.
Ok for trunk?
2018-12-11 Jakub Jelinek <jakub@redhat.com>
PR lto/86004
* lib/target-supports.exp (check_effective_target_lto_incremental):
New.
* g++.dg/lto/pr69137_0.C: Require lto_incremental effective target.
* g++.dg/lto/pr65316_0.C: Likewise.
* g++.dg/lto/pr85176_0.C: Likewise.
* g++.dg/lto/pr79000_0.C: Likewise.
* g++.dg/lto/pr66180_0.C: Likewise.
* g++.dg/lto/pr65193_0.C: Likewise.
* g++.dg/lto/pr69077_0.C: Likewise.
* g++.dg/lto/pr68057_0.C: Likewise.
* g++.dg/lto/pr66705_0.C: Likewise.
* g++.dg/lto/pr65302_0.C: Likewise.
* g++.dg/lto/20091002-1_0.C: Likewise.
* g++.dg/lto/pr81940_0.C: Likewise.
* g++.dg/lto/pr64043_0.C: Likewise.
* g++.dg/lto/pr65549_0.C: Likewise.
* g++.dg/lto/pr69133_0.C: Likewise.
* gfortran.dg/lto/pr79108_0.f90: Likewise.
--- gcc/testsuite/lib/target-supports.exp.jj 2018-12-04 20:40:01.491379395 +0100
+++ gcc/testsuite/lib/target-supports.exp 2018-12-10 15:39:37.714763670 +0100
@@ -8014,6 +8014,18 @@ proc check_effective_target_lto { } {
} "-flto"]
}
+# Return 1 if the compiler and linker support incremental link-time
+# optimization.
+
+proc check_effective_target_lto_incremental { } {
+ if ![check_effective_target_lto] {
+ return 0
+ }
+ return [check_no_compiler_messages lto_incremental executable {
+ int main () { return 0; }
+ } "-flto -r -nostdlib"]
+}
+
# Return 1 if -mx32 -maddress-mode=short can compile, 0 otherwise.
proc check_effective_target_maybe_x32 { } {
--- gcc/testsuite/g++.dg/lto/pr69137_0.C.jj 2016-01-15 20:37:28.533589631 +0100
+++ gcc/testsuite/g++.dg/lto/pr69137_0.C 2018-12-10 15:42:38.355806095 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -std=c++11 -g -flto } } }
// { dg-extra-ld-options "-r -nostdlib" }
--- gcc/testsuite/g++.dg/lto/pr65316_0.C.jj 2017-11-06 17:24:08.773286226 +0100
+++ gcc/testsuite/g++.dg/lto/pr65316_0.C 2018-12-10 15:41:01.092398551 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -flto -std=c++11 -g2 -fno-lto-odr-type-merging -O2 -Wno-return-type } } }
// { dg-extra-ld-options "-r -nostdlib -O2 -fno-lto-odr-type-merging" }
--- gcc/testsuite/g++.dg/lto/pr85176_0.C.jj 2018-04-04 16:13:42.972551086 +0200
+++ gcc/testsuite/g++.dg/lto/pr85176_0.C 2018-12-10 15:43:20.838110547 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -flto -g1 } } }
// { dg-extra-ld-options "-r -nostdlib" }
namespace a {
--- gcc/testsuite/g++.dg/lto/pr79000_0.C.jj 2017-01-09 11:35:04.310821717 +0100
+++ gcc/testsuite/g++.dg/lto/pr79000_0.C 2018-12-10 15:42:54.657539190 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { "-flto -g" } }
// { dg-extra-ld-options "-r -nostdlib" }
--- gcc/testsuite/g++.dg/lto/pr66180_0.C.jj 2015-05-29 15:04:33.064803028 +0200
+++ gcc/testsuite/g++.dg/lto/pr66180_0.C 2018-12-10 15:41:39.730765940 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -flto -std=c++14 -r -nostdlib } } }
#include <memory>
namespace {
--- gcc/testsuite/g++.dg/lto/pr65193_0.C.jj 2017-11-06 17:24:08.774286214 +0100
+++ gcc/testsuite/g++.dg/lto/pr65193_0.C 2018-12-10 15:40:23.564012993 +0100
@@ -1,5 +1,6 @@
/* { dg-lto-do link } */
/* { dg-require-effective-target fpic } */
+/* { dg-require-effective-target lto_incremental } */
/* { dg-lto-options {{-fPIC -r -nostdlib -flto -O2 -g -Wno-return-type}} } */
void frexp (int, int *);
--- gcc/testsuite/g++.dg/lto/pr69077_0.C.jj 2017-11-06 17:24:08.770286263 +0100
+++ gcc/testsuite/g++.dg/lto/pr69077_0.C 2018-12-10 15:42:19.886108489 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -O3 -g -flto } } }
// { dg-extra-ld-options "-r -nostdlib" }
--- gcc/testsuite/g++.dg/lto/pr68057_0.C.jj 2015-11-09 13:39:21.418394784 +0100
+++ gcc/testsuite/g++.dg/lto/pr68057_0.C 2018-12-10 15:42:08.058302145 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
/* { dg-extra-ld-options { -O2 -Wno-odr -r -nostdlib } } */
struct SPxPricer;
struct SoPlex {
--- gcc/testsuite/g++.dg/lto/pr66705_0.C.jj 2015-09-03 17:18:33.224711564 +0200
+++ gcc/testsuite/g++.dg/lto/pr66705_0.C 2018-12-10 15:41:53.763536188 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -O2 -flto -flto-partition=max -fipa-pta } } }
// { dg-extra-ld-options "-r -nostdlib" }
--- gcc/testsuite/g++.dg/lto/pr65302_0.C.jj 2017-11-06 17:24:08.770286263 +0100
+++ gcc/testsuite/g++.dg/lto/pr65302_0.C 2018-12-10 15:40:53.705519502 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -flto -O2 -Wno-return-type } } }
// { dg-extra-ld-options "-r -nostdlib -O0" }
--- gcc/testsuite/g++.dg/lto/20091002-1_0.C.jj 2017-11-06 17:24:08.768286288 +0100
+++ gcc/testsuite/g++.dg/lto/20091002-1_0.C 2018-12-10 15:40:03.968333822 +0100
@@ -1,5 +1,6 @@
// { dg-lto-do link }
// { dg-require-effective-target fpic }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options {{-fPIC -flto -Wno-return-type}} }
// { dg-extra-ld-options "-fPIC -r -nostdlib" }
--- gcc/testsuite/g++.dg/lto/pr81940_0.C.jj 2017-09-01 09:26:20.822938902 +0200
+++ gcc/testsuite/g++.dg/lto/pr81940_0.C 2018-12-10 15:43:07.350331378 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -O -flto } } }
// { dg-extra-ld-options "-r -nostdlib -g" }
--- gcc/testsuite/g++.dg/lto/pr64043_0.C.jj 2014-12-15 10:36:23.853604377 +0100
+++ gcc/testsuite/g++.dg/lto/pr64043_0.C 2018-12-10 15:26:51.800299063 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -flto -std=c++11 } } }
// { dg-extra-ld-options "-r -nostdlib -O2" }
class Validator
--- gcc/testsuite/g++.dg/lto/pr65549_0.C.jj 2017-11-06 17:24:08.773286226 +0100
+++ gcc/testsuite/g++.dg/lto/pr65549_0.C 2018-12-10 15:41:26.456983265 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -std=gnu++14 -flto -g -Wno-return-type } { -std=gnu++14 -flto -g -O2 -fno-inline -flto-partition=max -Wno-return-type } } }
// { dg-extra-ld-options "-r -nostdlib" }
--- gcc/testsuite/g++.dg/lto/pr69133_0.C.jj 2016-01-19 13:31:07.638760602 +0100
+++ gcc/testsuite/g++.dg/lto/pr69133_0.C 2018-12-10 15:42:30.451935499 +0100
@@ -1,4 +1,5 @@
// { dg-lto-do link }
+// { dg-require-effective-target lto_incremental }
// { dg-lto-options { { -flto -O2 } } }
// { dg-extra-ld-options "-r -nostdlib -flto -flto-partition=none -O2" }
namespace xercesc_3_1 {
--- gcc/testsuite/gfortran.dg/lto/pr79108_0.f90.jj 2018-09-21 19:07:12.649985507 +0200
+++ gcc/testsuite/gfortran.dg/lto/pr79108_0.f90 2018-12-10 15:43:49.724637597 +0100
@@ -1,4 +1,5 @@
! { dg-lto-do link }
+! { dg-require-effective-target lto_incremental }
! { dg-lto-options {{ -Ofast -flto --param ggc-min-expand=0 --param ggc-min-heapsize=0 }} }
! { dg-extra-ld-options "-r" }
Jakub
next reply other threads:[~2018-12-11 7:31 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-11 7:31 Jakub Jelinek [this message]
2018-12-11 7:53 ` Rainer Orth
2018-12-11 9:12 ` Jakub Jelinek
2018-12-11 9:41 ` Rainer Orth
2018-12-11 8:30 ` Richard Biener
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181211073122.GL12380@tucnak \
--to=jakub@redhat.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=jh@suse.cz \
--cc=rguenther@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).