mbox series

[0/3] Move some single-file LTO tests to torture

Message ID cover.1729893964.git.sam@gentoo.org
Headers show
Series Move some single-file LTO tests to torture | expand

Message

Sam James Oct. 25, 2024, 10:06 p.m. UTC
Andrew pointed this out when committing those testsuite fixes earlier. We
may as well make these proper torture tests rather than having them
unnecessarily in the special lto/ dir which is meant for multiple files
really.

Sam James (3):
  testsuite: move single-file LTO pr62026 test to torture
  testsuite: move single-file LTO pr95677 test to torture
  testsuite: move single-file LTO pr47333 test to torture

 gcc/testsuite/g++.dg/{lto/pr47333_0.C => torture/pr47333.C} | 3 +--
 gcc/testsuite/g++.dg/{lto/pr62026_0.C => torture/pr62026.C} | 4 ++--
 gcc/testsuite/g++.dg/{lto/pr95677_0.C => torture/pr95677.C} | 6 +-----
 3 files changed, 4 insertions(+), 9 deletions(-)
 rename gcc/testsuite/g++.dg/{lto/pr47333_0.C => torture/pr47333.C} (99%)
 rename gcc/testsuite/g++.dg/{lto/pr62026_0.C => torture/pr62026.C} (75%)
 rename gcc/testsuite/g++.dg/{lto/pr95677_0.C => torture/pr95677.C} (71%)

Comments

Sam James Oct. 31, 2024, 8:56 p.m. UTC | #1
Sam James <sam@gentoo.org> writes:

> Andrew pointed this out when committing those testsuite fixes earlier. We
> may as well make these proper torture tests rather than having them
> unnecessarily in the special lto/ dir which is meant for multiple files
> really.
>
> Sam James (3):
>   testsuite: move single-file LTO pr62026 test to torture
>   testsuite: move single-file LTO pr95677 test to torture
>   testsuite: move single-file LTO pr47333 test to torture
>

Do these look OK? I've dropped the pr62026 change locally because -r
doesn't play well with torture.

I think it's borderline obvious but it's somewhat opinionated so I'd
like someone to ack it. Thanks.

>  gcc/testsuite/g++.dg/{lto/pr47333_0.C => torture/pr47333.C} | 3 +--
>  gcc/testsuite/g++.dg/{lto/pr62026_0.C => torture/pr62026.C} | 4 ++--
>  gcc/testsuite/g++.dg/{lto/pr95677_0.C => torture/pr95677.C} | 6 +-----
>  3 files changed, 4 insertions(+), 9 deletions(-)
>  rename gcc/testsuite/g++.dg/{lto/pr47333_0.C => torture/pr47333.C} (99%)
>  rename gcc/testsuite/g++.dg/{lto/pr62026_0.C => torture/pr62026.C} (75%)
>  rename gcc/testsuite/g++.dg/{lto/pr95677_0.C => torture/pr95677.C} (71%)
Mike Stump Oct. 31, 2024, 10:16 p.m. UTC | #2
On Oct 31, 2024, at 1:56 PM, Sam James <sam@gentoo.org> wrote:
> 
> Sam James <sam@gentoo.org> writes:
> 
>> Andrew pointed this out when committing those testsuite fixes earlier. We
>> may as well make these proper torture tests rather than having them
>> unnecessarily in the special lto/ dir which is meant for multiple files
>> really.
>> 
>> Sam James (3):
>>  testsuite: move single-file LTO pr62026 test to torture
>>  testsuite: move single-file LTO pr95677 test to torture
>>  testsuite: move single-file LTO pr47333 test to torture
>> 
> 
> Do these look OK?

Ok.