Re: [PATCH 2/2] xdiff: optimize xdl_hash_record_verbatim
From: Alexander Monakov <hidden>
Date: 2025-08-12 17:56:46
On Mon, 11 Aug 2025, Phillip Wood wrote:quoted
quoted
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