On Tue, 16 Jul 2024, Richard Biener wrote:

> On Tue, 16 Jul 2024, Jakub Jelinek wrote:
> 
> > On Tue, Jul 16, 2024 at 10:55:30AM +0200, Richard Biener wrote:
> > > On Tue, 16 Jul 2024, Jakub Jelinek wrote:
> > > 
> > > > On Tue, Jul 16, 2024 at 10:43:27AM +0200, Richard Biener wrote:
> > > > > I've pushed it to trunk now and am running local CPU 2017 to check for
> > > > > obvious fallout on Zen4 so we can make 14.2 RC early next week.  
> > > > > There's
> > > > > still the question of GCC 11.5 which got the backport of zen4 support
> > > > > with this "wrong" costs but RC1 was already last week and we're set
> > > > > to release on Friday.
> > > > > 
> > > > > I'd like to hear your opinion on that (13.3 and 12.4 also got the 
> > > > > bogus
> > > > > value so eventually 11.5 getting the bogus value isn't too bad).
> > > > > 
> > > > > Btw, I just see that znver5 tables have the same issue, I'll push the
> > > > > obvious change there as well.
> > > > 
> > > > I'd just change 11.5 and do a RC2.
> > > 
> > > I'm undecided on how important it is or whether it would need a RC, I 
> > > don't expect anybody to use -march=znver4 with a GCC 11 code-base since
> > > it wasn't available there with 11.4.
> > 
> > -march=native is something people will actually use even with GCC 11 code
> > base and I've just tested that on znver4 box
> > ~/xgcc -B ~/ -S -march=native -xc /dev/null -o /dev/null -v 2>&1 | sed -n 
> > 's/^.* \(-march=[0-9a-z]*\) .*$/\1/p'
> > -march=znver4
> > ~/xgcc --version
> > xgcc (GCC) 11.4.1 20240620
> > Copyright (C) 2021 Free Software Foundation, Inc.
> > This is free software; see the source for copying conditions.  There is NO
> > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> > 
> > So, I think we really should fix this for 11.5.
> 
> OK, note we have the broken state in 12.4 and 13.3 so that got some 
> testing there.  I actually expect less problems with the fixed values
> as the bogus ones will cause alignment peeling that's not seen before
> with -march=native, possibly triggering latent issues like PR115841
> (not that in particular since GCC 11 isn't affected).
> 
> I'll push the cost adjustment to 11 later but will wait with the other
> branches to watch SPEC results as Honza requested.

Pushed to the GCC 11 branch now, after bootstrapping and testing on
x86_64-unknown-linux-gnu.

Do you think this needs a new RC?

Richard.

Reply via email to