Jiufu Guo <guoji...@linux.ibm.com> writes:

> Segher Boessenkool <seg...@kernel.crashing.org> writes:
>
>> Hi!
>>
[...]
>>> --- a/gcc/testsuite/gcc.dg/torture/stackalign/builtin-return-1.c
>>> +++ b/gcc/testsuite/gcc.dg/torture/stackalign/builtin-return-1.c
>>> @@ -2,6 +2,7 @@
>>>  /* Originator: Andrew Church <gcczi...@achurch.org> */
>>>  /* { dg-do run } */
>>>  /* { dg-require-effective-target untyped_assembly } */
>>> +/* { dg-additional-options "-fno-rename-registers" { target { powerpc*-*-* 
>>> } } } */
>>
>> What is this for?  What happens without it?
> The reason of this fail is: -frename-registers does not work well with
> __builtin_return/__builtin_apply which need to save and restore
> registers which could not be renamed incorrectly.
For this issue, I opened a bug
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93047.

Thanks,
Jiufu.

>
> When this case runs with -O3, with this patch, -frename-registers is
> enabled. Originally, -frename-registers is enabled with -funroll-loops
> instead pure -O3. This change cause this case fail at -O3.
>
>>
>> The rs6000/ parts are okay for trunk.  Thanks!
>>
>>
>> Segher

Reply via email to