On Thu, 3 Jun 2021 18:40:09 GMT, Sandhya Viswanathan <sviswanat...@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
