Hi Jakub,
> -----Original Message-----
> From: Jakub Jelinek <ja...@redhat.com>
> Sent: Tuesday, March 27, 2018 4:43 PM
> To: Kumar, Venkataramanan <venkataramanan.ku...@amd.com>
> Cc: Richard Biener <rguent...@suse.de>; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Improve TRUTH_{AND,OR}IF_EXPR expansion (PR rtl-
> optimization/78200)
>
> 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?
Yes, we can mark this PR as GCC 7 regression only.
There is another PR (84481) for 429.mcf on Zen regression against 7.x which
seem to be independent of this issue.
>
> Jakub
