RFR: 8266054: VectorAPI rotate operation optimization [v2]
Jatin Bhateja
jbhateja at openjdk.java.net
Mon May 3 06:51:37 UTC 2021
On Fri, 30 Apr 2021 15:44:41 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:
>> Jatin Bhateja has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>>
>> - 8266054: Review comments resolution.
>> - Merge http://github.com/openjdk/jdk into JDK-8266054
>> - 8266054: Changing gen-src.sh file permissions
>> - 8266054: VectorAPI rotate operation optimization
>
> src/hotspot/cpu/x86/x86.ad line 1652:
>
>> 1650: case Op_RotateRightV:
>> 1651: case Op_RotateLeftV:
>> 1652: if (is_subword_type(bt)) {
>
> Does that have the effect of not intrinsifying for `byte` or `short`?
Yes, it makes sure that intrinsification is based on Shifts and Or operations instead of Rotation operation.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3720
More information about the core-libs-dev
mailing list