[Bug testsuite/92398] [10 regression] error in update of gcc.target/powerpc/pr72804.c in r277872

bergner at gcc dot gnu.org gcc-bugzilla@gcc.gnu.org
Tue Nov 12 15:37:00 GMT 2019


https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92398

Peter Bergner <bergner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bergner at gcc dot gnu.org

--- Comment #4 from Peter Bergner <bergner at gcc dot gnu.org> ---
(In reply to Xiong Hu XS Luo from comment #2)
> Sorry that I mistook to change the case pr72804.c: this case has no
> relationship with the parameter -fno-inline-functions --param
> max-inline-insns-single-O2=200.  This test appears in category of "New tests
> that FAIL (6 tests):".

Correct, I don't think -fno-inline-functions --param
max-inline-insns-single-O2=200 should have ever been added to pr72804.c.  The
correct thing would be to remove those options from the test case.

However, I don't see how those options could be causing any change in generated
assembly because there are no inlining opportunities, so how could that
revision be causing the reported error?  Bill, are you sure on that revision?



> Peter added this case pr72804.c in r251153 of Aug 17, 2017 to generate
> better code for -mvsx-timode. But it restrict the condition to
> !BYTES_BIG_ENDIAN and !TARGET_P9_VECTOR:

I added this test to test for poor code generation on POWER7 (and POWER8). 
Looking at the POWER9 output, that code looks good to me.  Maybe we didn't
generate that back when I added the test case?  The POWER8 (BE) code does look
worse to me than what we expect.

Bill, can you reconfirm what revision broke pr72804.c?


More information about the Gcc-bugs mailing list