On Tue, 11 Apr 2023 19:03:21 GMT, Vladimir Ivanov <[email protected]> wrote:
>> Quan Anh Mai has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> style
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/ShortVector.java
> line 2295:
>
>> 2293: // to be performant
>> 2294: @ForceInline
>> 2295: public ShortVector apply(ShortVector v1, ShortVector
>> v2, int o) {
>
> Have you considered matching the corresponding IR during GVN to produce
> VectorSlice nodes rather than going through VM intrinsic?
I have thought about this but it will require C2 to track the values of
individual elements in a vector and constant fold vector loads from stable
fields, both of which are not available as of right now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12909#discussion_r1186028378
