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