Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Richard Biener
On Tue, 16 Jul 2024, Richard Biener wrote: > On Mon, 15 Jul 2024, Jan Hubicka wrote: > > > > Currently unaligned YMM and ZMM load and store costs are cheaper than > > > aligned which causes the vectorizer to purposely mis-align accesses > > > by adding an alignment prologue. It looks like the un

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Richard Biener
On Tue, 16 Jul 2024, Jakub Jelinek wrote: > On Tue, Jul 16, 2024 at 01:04:50PM +0200, Richard Biener wrote: > > Do you think this needs a new RC? > > Guess that depends on if somebody would actually perform testing with that > option... ... on a Zen4 machine. I think that's unlikely, let's see

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Jakub Jelinek
On Tue, Jul 16, 2024 at 01:04:50PM +0200, Richard Biener wrote: > Do you think this needs a new RC? Guess that depends on if somebody would actually perform testing with that option... Jakub

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Richard Biener
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'v

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Richard Biener
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

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Jakub Jelinek
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 mak

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Richard Biener
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 whi

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Jakub Jelinek
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 "wr

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-16 Thread Richard Biener
On Mon, 15 Jul 2024, Jan Hubicka wrote: > > Currently unaligned YMM and ZMM load and store costs are cheaper than > > aligned which causes the vectorizer to purposely mis-align accesses > > by adding an alignment prologue. It looks like the unaligned costs > > were simply left untouched from znve

Re: [PATCH] Fixup unaligned load/store cost for znver4

2024-07-15 Thread Jan Hubicka
> Currently unaligned YMM and ZMM load and store costs are cheaper than > aligned which causes the vectorizer to purposely mis-align accesses > by adding an alignment prologue. It looks like the unaligned costs > were simply left untouched from znver3 where they equate the aligned > costs when twe