[vectorIntrinsics+mask] RFR: 8264563: Add masked vector intrinsics for binary/store operations [v5]

Xiaohong Gong xgong at openjdk.java.net
Thu Apr 15 02:18:48 UTC 2021


On Wed, 14 Apr 2021 20:11:30 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:

>> Xiaohong Gong has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove null checking for the mask argument in API implementation
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/IntVector.java line 640:
> 
>> 638:             }
>> 639:             // suppress div/0 exceptions in unset lanes
>> 640:             that = that.lanewise(NOT, eqz);
> 
> I guess this is required because we don't know how the intrinsic will support masking? How can a  div/0 exception can occur?

Thanks for looking at this PR @PaulSandoz ! Yes, this is actually required for masked division operation. I think it has to make sure no div/0 exception happens during hotspot compiling the intrinsic? Am I missing something? Thanks!

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

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


More information about the panama-dev mailing list