* [PATCH] c++: testsuite: require lto_incremental in pr90990_0.C
@ 2022-06-21 5:57 Alexandre Oliva
2022-06-21 6:59 ` Richard Biener
0 siblings, 1 reply; 2+ messages in thread
From: Alexandre Oliva @ 2022-06-21 5:57 UTC (permalink / raw)
To: gcc-patches; +Cc: Richard Biener
Other LTO tests that use -r require the lto_incremental effective
target. I suppose pr90990_0.C is missing it due to an oversight.
This patch arranges for this test to also be skipped on
non-lto_incremental targets.
Regstrapped on x86_64-linux-gnu, also tested with a cross to
aarch64-rtems6. Ok to install?
for gcc/testsuite/ChangeLog
* g++.dg/lto/pr90990_0.C: Require lto_incremental target.
---
gcc/testsuite/g++.dg/lto/pr90990_0.C | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/testsuite/g++.dg/lto/pr90990_0.C b/gcc/testsuite/g++.dg/lto/pr90990_0.C
index 22a5e3ffaaa45..74cc2bbd92889 100644
--- a/gcc/testsuite/g++.dg/lto/pr90990_0.C
+++ b/gcc/testsuite/g++.dg/lto/pr90990_0.C
@@ -1,5 +1,6 @@
// { dg-lto-do link }
/* { dg-extra-ld-options { -r -nostdlib } } */
+// { dg-require-effective-target lto_incremental }
class A {
public:
float m_floats;
--
Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/
Free Software Activist GNU Toolchain Engineer
Disinformation flourishes because many people care deeply about injustice
but very few check the facts. Ask me about <https://stallmansupport.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] c++: testsuite: require lto_incremental in pr90990_0.C
2022-06-21 5:57 [PATCH] c++: testsuite: require lto_incremental in pr90990_0.C Alexandre Oliva
@ 2022-06-21 6:59 ` Richard Biener
0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2022-06-21 6:59 UTC (permalink / raw)
To: Alexandre Oliva; +Cc: gcc-patches
On Tue, 21 Jun 2022, Alexandre Oliva wrote:
>
> Other LTO tests that use -r require the lto_incremental effective
> target. I suppose pr90990_0.C is missing it due to an oversight.
> This patch arranges for this test to also be skipped on
> non-lto_incremental targets.
>
> Regstrapped on x86_64-linux-gnu, also tested with a cross to
> aarch64-rtems6. Ok to install?
OK.
>
> for gcc/testsuite/ChangeLog
>
> * g++.dg/lto/pr90990_0.C: Require lto_incremental target.
> ---
> gcc/testsuite/g++.dg/lto/pr90990_0.C | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/gcc/testsuite/g++.dg/lto/pr90990_0.C b/gcc/testsuite/g++.dg/lto/pr90990_0.C
> index 22a5e3ffaaa45..74cc2bbd92889 100644
> --- a/gcc/testsuite/g++.dg/lto/pr90990_0.C
> +++ b/gcc/testsuite/g++.dg/lto/pr90990_0.C
> @@ -1,5 +1,6 @@
> // { dg-lto-do link }
> /* { dg-extra-ld-options { -r -nostdlib } } */
> +// { dg-require-effective-target lto_incremental }
> class A {
> public:
> float m_floats;
>
>
--
Richard Biener <rguenther@suse.de>
SUSE Software Solutions Germany GmbH, Frankenstraße 146, 90461 Nuernberg,
Germany; GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman;
HRB 36809 (AG Nuernberg)
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-06-21 6:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-21 5:57 [PATCH] c++: testsuite: require lto_incremental in pr90990_0.C Alexandre Oliva
2022-06-21 6:59 ` Richard Biener
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).