RFR: 8310691: [REDO] [vectorapi] Refactor VectorShuffle implementation [v2]
Paul Sandoz
psandoz at openjdk.org
Fri Dec 6 16:23:40 UTC 2024
On Thu, 5 Dec 2024 20:36:48 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:
>> The cast is added so that we have the concrete type of the shuffle, the result of `toShuffle` is only `VectorShuffle<Integer>`
>
> Ah i see now, you want to ensure an invocation to the final/concrete method. (The IDE's highlighting of the redundant cast is misleading)
The common way we tend to do this in other areas is assign to a local variable with the sharper type. This makes it a littler clearer on the intent. Could you do that?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21042#discussion_r1873652324
More information about the hotspot-compiler-dev
mailing list