Webrev for vectorIntrinsics branch

Lupusoru, Razvan A razvan.a.lupusoru at intel.com
Fri Dec 15 00:45:08 UTC 2017


Hi Henry,

Sounds good to me - please go ahead with the branch creation!

Also, thank you again for doing this and also being willing to help with some of the additional patch porting.

--Razvan


-----Original Message-----
From: Henry Jen [mailto:henry.jen at oracle.com] 
Sent: Wednesday, December 13, 2017 11:10 AM
To: Lupusoru, Razvan A <razvan.a.lupusoru at intel.com>
Cc: panama-dev at openjdk.java.net
Subject: Re: Webrev for vectorIntrinsics branch



> On Dec 13, 2017, at 8:51 AM, Lupusoru, Razvan A <razvan.a.lupusoru at intel.com> wrote:
> 
> Thank you for doing this Henry!
> 
> You are right - what is under vector-draft should be the code that was migrated to become jdk.incubator.vector. And also it looks like you have the correct patches circa October.
> 
> Two questions:
> - Can you tell us the titles of the patches that you merged together? This will help keep track when looking back to find if everything was included.

It should have everything relevant up to following patch in JDK and hotspot,

Double256Vector JDK support for VectorAPI Implementation


> - Will you or someone else be helping with the port of all other Vector patches to the new branch?
> 

Yes, I can help with that once this initial webrev for vectorIntrinsics branch pushed.

Cheers,
Henry


> Thanks!
> --Razvan
> 
> -----Original Message-----
> From: panama-dev [mailto:panama-dev-bounces at openjdk.java.net] On Behalf Of Henry Jen
> Sent: Tuesday, December 12, 2017 11:56 PM
> To: panama-dev at openjdk.java.net
> Subject: RFR: Webrev for vectorIntrinsics branch
> 
> Hi,
> 
> I prepared a webrev for what I think belongs to vectorIntrinsics branch, which is to host intrinsic-based vector API implementation. You can review the webrev at http://cr.openjdk.java.net/~henryjen/panama/vectorIntrinsics/webrev/
> 
> This is base on the snapshot we took from panama/panama on beginning of October, there are quite a few commits after this, which we should be able to apply on top of this repo later.
> 
> Basically it should contains jdk.incubator.vector and hotspot changes for C2 intrinsics. If I understand it correctly, what’s under test/panama/vector-draft-spec are now jdk.incubator.vector, so that can be removed.
> 
> We will have another vectorSnippets branch to host snippet-based vector API implementation, that should have the rest under test/panama/ except vector-draft-spec.
> 
> Please correct me if above understanding doesn’t make sense to you or any issue in the webrev.
> 
> Cheers,
> Henry
> 



More information about the panama-dev mailing list