On Wed, 28 Aug 2024 13:16:03 GMT, Shaojin Wen <s...@openjdk.org> wrote:

> A small optimization makes BufWriterImpl's writeU1/U2/Int/Long methods more 
> C2-friendly and improves performance.

Ok, I did some adhoc testing to convince me that these are equivalent for these 
cases. Masking in line with intended semantics doesn't add any real runtime 
cost, though, it's mainly a declaration of intent, but if you prefer it like 
this then go ahead.

-------------

Marked as reviewed by redestad (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20748#pullrequestreview-2268614060

Reply via email to