[vectorIntrinsics+fp16] RFR: 8290872: Extend panama-vector test automation for FP16 support [v5]
Swati Sharma
duke at openjdk.org
Mon Aug 22 10:37:46 UTC 2022
On Wed, 17 Aug 2022 06:39:32 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:
>> Swati Sharma has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8290872: Resolved review comments.
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/AbstractSpecies.java line 649:
>
>> 647: }
>> 648: // Let the assert condition true for Halffloat
>> 649: assert(s.laneType == laneType) || laneType.switchKey == LaneType.SK_HALFFLOAT : s + "!=" + laneType;
>
> Is it better if:
>
>
> if (laneType.switchKey != LaneType.SK_HALFFLOAT) {
> assert(s.laneType == laneType) : s + "!=" + laneType;
> }
With this approach an extra check will be required for each lane type, which can be avoided by OR short circuiting.
-------------
PR: https://git.openjdk.org/panama-vector/pull/205
More information about the panama-dev
mailing list