Re: [PATCH 2/2] i386: correct x87 multiplication modeling in znver.md

2022-11-16 Thread Jan Hubička via Gcc-patches
On Wed, Nov 16, 2022 at 2:13 PM Alexander Monakov wrote: > > On Wed, 16 Nov 2022, Jan Hubička wrote: > > > This looks really promising. I will experiment with the patch for > separate > > znver3 model, but I think we should be able to keep > > them unified and hopefully get both less code

Re: [PATCH 2/2] i386: correct x87 multiplication modeling in znver.md

2022-11-16 Thread Alexander Monakov via Gcc-patches
On Wed, 16 Nov 2022, Jan Hubička wrote: > This looks really promising. I will experiment with the patch for separate > znver3 model, but I think we should be able to keep > them unified and hopefully get both less code duplicatoin and table sizes. Do you mean separate znver4 (not '3') model

Re: [PATCH 2/2] i386: correct x87 multiplication modeling in znver.md

2022-11-16 Thread Jan Hubička via Gcc-patches
Hello, On Wed, Nov 16, 2022 at 12:53 PM Kumar, Venkataramanan < venkataramanan.ku...@amd.com> wrote: > [AMD Official Use Only - General] > > Hi, > > > > Top znver table sizes in insn-automata.o: > > > > Before: > > > > 30056 r znver1_fp_min_issue_delay > > 120224 r znver1_fp_transitions > > > >

RE: [PATCH 2/2] i386: correct x87 multiplication modeling in znver.md

2022-11-16 Thread Kumar, Venkataramanan via Gcc-patches
Monakov > > Subject: [PATCH 2/2] i386: correct x87 multiplication modeling in > znver.md > > Caution: This message originated from an External Source. Use proper > caution when opening attachments, clicking links, or responding. > > > All multiplication instructions a

[PATCH 2/2] i386: correct x87 multiplication modeling in znver.md

2022-11-01 Thread Alexander Monakov
All multiplication instructions are fully pipelined, except AVX256 instructions on Zen 1, which issue over two cycles on a 128-bit unit. Correct the model accordingly to reduce combinatorial explosion in automaton tables. Top znver table sizes in insn-automata.o: Before: 30056 r