RFR: 8268151: Vector API toShuffle optimization [v2]

Xiaohong Gong xgong at openjdk.java.net
Fri Jun 4 03:37:02 UTC 2021


On Thu, 3 Jun 2021 18:40:09 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> src/hotspot/share/opto/vectornode.cpp line 1246:
>> 
>>> 1244:           return new VectorLoadMaskNode(value, out_vt);
>>> 1245:         } else if (is_vector_shuffle) {
>>> 1246:           if (!is_shuffle_to_vector()) {
>> 
>> Hi @sviswa7 , thanks for this change! I'm just curious whether `is_shuffle_to_vector()` is still needed for `VectorUnboxNode` with this change? It seems this flag can be removed, doesn't it?
>
> @XiaohongGong is_shuffle_to_vector is still needed as we shouldn't generate VectorLoadShuffleNode for shuffle.toVector.

OK, got it. Thanks!

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

PR: https://git.openjdk.java.net/jdk/pull/4326


More information about the core-libs-dev mailing list