On Thu, Aug 02, 2018 at 09:02:05PM -0500, Siddhesh Poyarekar wrote:
> On 08/03/2018 12:02 AM, James Greenhalgh wrote:
> > On Thu, Aug 02, 2018 at 11:58:37AM -0500, Siddhesh Poyarekar wrote:
> >> There was a typo in the pipeline description where DUP was assigned to
> >> the vector pipes for quad mode ops when it really only uses the VTOG
> >> pipes.  Fixing this does not show any noticeable difference in
> >> performance (there's a very small bump of 1.7% in x264 but that's
> >> about it) in my tests but is the more precise description of operations
> >> for falkor.
> >>
> >> Bootstrapped and tested with --with-cpu=falkor to confirm that there
> >> are no regressions.
> > 
> > OK.
> 
> Thanks, may I backport this to gcc 8.x as well given that it has zero 
> impact on anything except falkor?  We are targeting gcc 8.x as the 
> toolchain for falkor and so would like to at least get all falkor 
> specific optimizations and fixes back into the gcc 8.x branch.

This one is OK for backport.

Thanks,
James

Reply via email to