Re: [PATCH 2/2] xdiff: optimize xdl_hash_record_verbatim
- From
Alexander Monakov <amonakov@ispras.ru>
- Date
- Aug 12, 2025, 17:56 UTC
- Message-ID
- <0379ba2d-837b-761e-9d5a-d65ca9d051d6@ispras.ru>
- In-Reply-To
- <c2fe3b69-8436-af46-c47d-dde5bb037227@ispras.ru>
Show 16 quoted lines
> On Mon, 11 Aug 2025, Phillip Wood wrote: > > > > That's what the 'cycles' column in the table gives (6.21/5.8 = 1.070...) > > > > It would be helpful to add a column with those calculations in it rather than > > forcing the reader to calculate the speed up for themselves. > > Ok, will change it to > > version | speedup over (A) | cycles, bn | instructions, bn > ---------------------------------------------------------- > A 6.38 11.3 > B 1.027 6.21 10.89 > C 1.1 5.80 9.95 > D 1.094 5.83 8.74 > ----------------------------------------------------------
On my Skylake:
version | speedup over (A) | cycles, bn | instructions, bn ---------------------------------------------------------- A 5.77 10.96 B 1.076 5.36 10.60 C 1.12 5.16 9.66 ----------------------------------------------------------
A is today's master, B and C are patch 1 and 1+2 like before.
Alexander