On Mon, Aug 27, 2018 at 10:05:17AM -0500, Luis Machado wrote: > Hi, > > On 08/08/2018 04:54 AM, Siddhesh Poyarekar wrote: > > On 08/01/2018 04:23 AM, James Greenhalgh wrote: > >> On Wed, Jul 25, 2018 at 01:10:34PM -0500, Luis Machado wrote: > >>> The adjusted vector costs give Falkor a reasonable boost in > >>> performance for FP > >>> benchmarks (both CPU2017 and CPU2006) and doesn't change INT > >>> benchmarks that > >>> much. About 0.7% for CPU2017 FP and 1.54% for CPU2006 FP. > >>> > >>> OK for trunk? > >> > >> OK if this is what works best for your subtarget. > >> > > > > I have pushed this for Luis since he is on holiday. > > > > Thanks, > > Siddhesh > > Given this has a significant performance impact on Falkor and the > changes are very localized and specific to Falkor, it would be great to > have them on GCC 8. Would it be ok to push it?
I've not got a problem with this if your opinion is that the risk is worthwhile for Falkor. Thanks, James