On Fri, 19 Sept 2025 at 14:11, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
> This patch uses left/right shift operators to optimize *2 and /2 operations in some functions that are in critical paths.
>
> For unsigned int, *2 and /2 exactly equal to <<1 and >>1.
The compiler is most likely going to do this anyway. Try it out at
https://godbolt.org/z/Y538Yd4Ka
What maybe is worth looking at is verifying which of these variables
is signed when it really should be unsigned. Dividing by 2 and >> 1
aren't the same for negative numbers.
David