fastxor icon indicating copy to clipboard operation
fastxor copied to clipboard

way to make it a bit faster

Open jwinkler2083233 opened this issue 4 years ago • 1 comments

Thanks for making this.

I noticed that you're putting the comparison at the top of the assembly functions. If you put the comparisons at the end, it would be a bit faster. You can assume that there is at least one element.

It's like using 'for ()' when you could instead use 'do...while{}'

jwinkler2083233 avatar Aug 28 '21 18:08 jwinkler2083233

That would save what, 1 CMP? Doesn't seem worth it to me. But if you make a PR and the benchmark shows an improvement, I'll merge it.

lukechampine avatar Aug 28 '21 23:08 lukechampine