On Sun, 4 Aug 2024 at 02:11, David Rowley <dgrowle...@gmail.com> wrote: > I did some more testing on this on a few different machines; apple M2 > Ultra, AMD 7945HX and with a Raspberry Pi 4.
I did some more testing on this patch today as I wanted to see what Intel CPUs thought about it. The only modern Intel CPU I have is a 13th-generation laptop CPU. It's an i7-1370P. It's in a laptop with solid-state cooling. At least, I've never heard a fan running on it. Watching the clock speed during the test had it jumping around wildly, so I assume it was thermally throttling. I've attached the results here anyway. They're very noisy. I also did a test where I removed all the escaping logic and had the code copy the source string to the destination without checking for chars to escape. I wanted to see how much was left performance-wise. There was only a further 10% increase. I tidied up the patch a bit more and pushed it. Thanks for the reviews. David
