On Wed, 17 Jun 2026 19:58:03 GMT, Andrew Dinn <[email protected]> wrote:

>> @adinn : Well, I have changed it, but that caused a small, but consistent 
>> performance drop (~0.7%) in the benchmark on the M1 chip that I measured it 
>> on. Since this PR is for performance, I am inclined to revert this change, 
>> if you agree.
>
> Hmm, interesting result. Yes, please stick with the current version. But also 
> update the comment to note that this lsl;lsr;orr sequence out-performs 
> extract.

... on one particular implementation. Let's not forget that we're writing for 
the architecture.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30941#discussion_r3434112064

Reply via email to