[vectorIntrinsics] RFR: 8274533: [Vector API] Incorrect computation by VectorMask.toLong operation after JDK-8273949

[Jatin Bhateja]

On Thu, 30 Sep 2021 11:43:51 GMT, Jie Fu <jiefu at openjdk.org> wrote:

>> Hi Jei, 
>> I am working on an alternate patch which fixes instruction sequence in legacy storeMask patterns.
>> Will share it shortly, I think it will also address this issue, kindly review the patch once its posted.
>> 
>> Best Regards,
>> Jatin
>
>> Hi Jei, I am working on an alternate patch which fixes instruction sequence in legacy storeMask patterns. Will share it shortly, I think it will also address this issue, kindly review the patch once its posted.
>> 
>> Best Regards, Jatin
> 
> Hi @jatin-bhateja ,
> 
> This is just a small bug fix, which follows the code style of current implementation.
> Maybe you can still go on with your enhancement after this fix.
> 
> It took me hours to narrow down and fix the problem.
> Otherwise, it would be completely a waste of time to do this kind of things.
> Thanks.

Hi @DamonFool , I have created a pull request which tries to address this issues in a differ manner. 
Please take a look. 
For your current patch since its a fix in AVX2 pattern to masklen will always be less than 64 in all the cases.

-------------

PR: https://git.openjdk.java.net/panama-vector/pull/138