Extract/Insert Java API Modifications for VectorAPI

Rukmannagari, Shravya shravya.rukmannagari at intel.com
Tue Mar 31 23:45:07 UTC 2020


Hi Paul,
Please find the webrev below which leaves the existing definition of extract/insert for *MaxVector as is. 
http://cr.openjdk.java.net/~srukmannagar/VectorAPI_vectorExtractInsert/webrev.02/

Thanks,
Shravya.

-----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.
>> 
> 



More information about the panama-dev mailing list