Extract/Insert Java API Modifications for VectorAPI

[Vladimir Ivanov]

> http://cr.openjdk.java.net/~srukmannagar/VectorAPI_vectorExtractInsert/webrev.02/

+        switch (i) {
+            case 0: return VectorSupport.insert(VCLASS, ETYPE, VLENGTH, 
this, 0, (long)e, (v, ix, bits) -> { int[] res = v.vec().clone(); 
res[ix] = (int)bits; return v.vectorFactory(res); });
+            case 1: return VectorSupport.insert(VCLASS, ETYPE, VLENGTH, 
this, 1, (long)e, (v, ix, bits) -> { int[] res = v.vec().clone(); 
res[ix] = (int)bits; return v.vectorFactory(res); });
+            default: throw new InternalError();
+        }

What about introducing a helper method, put the intrinsic call there, 
and call it with different indices from the switch?

It should significantly reduce amount of duplicated code.

Best regards,
Vladimir Ivanov

> -----Original Message-----
> From: Paul Sandoz <paul.sandoz at oracle.com>
> Sent: Tuesday, March 31, 2020 2:48 PM
> To: Rukmannagari, Shravya <shravya.rukmannagari at intel.com>
> Cc: panama-dev <panama-dev at openjdk.java.net>
> Subject: Re: Extract/Insert Java API Modifications for VectorAPI
> 
> Can we leave the existing code in place for *MaxVector?
> 
> Paul.
> 
>> On Mar 31, 2020, at 2:06 PM, Rukmannagari, Shravya <shravya.rukmannagari at intel.com> wrote:
>>
>> Hi Paul,
>> Thanks a lot for the review. There seems to be no easy solution to handle MaxVector with the current patch. We can drop this for now and handle it in the future updates.
>>
>> Thanks,
>> Shravya.
>>
>> -----Original Message-----
>> From: Paul Sandoz <paul.sandoz at oracle.com>
>> Sent: Tuesday, March 31, 2020 1:04 PM
>> To: Rukmannagari, Shravya <shravya.rukmannagari at intel.com>
>> Cc: panama-dev <panama-dev at openjdk.java.net>
>> Subject: Re: Extract/Insert Java API Modifications for VectorAPI
>>
>> Hi,
>>
>> Will the Java implementation fail for *MaxVector for lanes > 8?
>>
>> Reducing the horizontal line-length would help for review purposes.  Perhaps place the lambda argument at the end on a new line.
>>
>> Paul.
>>
>>> On Mar 31, 2020, at 12:40 PM, Rukmannagari, Shravya <shravya.rukmannagari at intel.com> wrote:
>>>
>>> Hi All,
>>> Please find the webrev below which modifies the definition of extract and insert. These are currently intrinsified on x86 platforms for a constant value idx, whereas the API provides a runtime variable to the function call.
>>> http://cr.openjdk.java.net/~srukmannagar/VectorAPI_vectorExtractInsert/webrev.01/
>>>
>>> Thanks,
>>> Shravya.
>>>
>>
>