Re: [committed] [PR105032] LRA: modify loop condition to find reload insns for hard reg splitting

[Vladimir Makarov <vmakarov@redhat.com>]

On 2022-03-30 15:18, Uros Bizjak wrote:
> On Wed, Mar 30, 2022 at 7:15 PM Vladimir Makarov via Gcc-patches
> <gcc-patches@gcc.gnu.org> wrote:
>> The following patch fixes
>>
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105032
>>
>> The patch was successfully bootstrapped and tested on x86-64.
> diff --git a/gcc/testsuite/gcc.target/i386/pr105032.c
> b/gcc/testsuite/gcc.target/i386/pr105032.c
> new file mode 100644
> index 00000000000..57b21d3cd7a
> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/i386/pr105032.c
> @@ -0,0 +1,36 @@
> +/* { dg-do compile } */
> +/* { dg-options "-w" } */
> +/* { dg-additional-options "-m32" { target x86_64-*-* } } */
>
> Please don't use -m32 in options, but instead conditionally compile
> the testcase with

Sorry for may be a stupid question.  I am interesting what are the 
reasons for this.  Is it just for saving computer cycles?

I think the test is important therefore I'd like to run the test on 
x86-64 too because people rarely test i686 target.

> /* { dg-do compile { target ia32 } } */