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