This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Improve TRUTH_{AND,OR}IF_EXPR expansion (PR rtl-optimization/78200)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: "Kumar, Venkataramanan" <Venkataramanan dot Kumar at amd dot com>
- Cc: Richard Biener <rguenther at suse dot de>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 27 Mar 2018 13:12:40 +0200
- Subject: Re: [PATCH] Improve TRUTH_{AND,OR}IF_EXPR expansion (PR rtl-optimization/78200)
- References: <20180327091023.GW8577@tucnak> <CY4PR12MB1736779C2772272BBAD5FDF38FAC0@CY4PR12MB1736.namprd12.prod.outlook.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Mar 27, 2018 at 11:04:35AM +0000, Kumar, Venkataramanan wrote:
> > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk if it
> > helps for SPEC? Venkat, do you think you could benchmark it in the setup
> > where you've measured the slowdown to see if it helps? I see the patch
> > changes the loop:
Thanks for benchmarking it.
> The patch causes regression benchmark when I measured on my Ryzen box (>4%) .
>
> GCC trunk: -O3 -maxv2 -mprefer-avx128 -O3 -march=znver1
> 429.mcf 9120 238 38.3 * 9120 227 40.2 *
>
> GCC patch:
> 429.mcf 9120 251 36.3 * 9120 236 38.6 *
So, has 429.mcf improved then compared to 7.x sufficiently that we can turn
PR78200 into just [7 Regression], without adding any patch?
Jakub