On Fri, 30 Apr 2021 15:44:41 GMT, Paul Sandoz <psan...@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