RFR: 8356760: VectorAPI: Optimize VectorMask.fromLong for all-true/all-false cases [v4]

erifan duke at openjdk.org
Thu Jul 24 03:41:55 UTC 2025


On Tue, 22 Jul 2025 09:04:56 GMT, erifan <duke at openjdk.org> wrote:

>>> Do you mean this check `Matcher::match_rule_supported_vector(opc, vlen, maskall_bt)` ? I think it's necessary ? Because in theory some platforms don't support both `MaskAll` and `Replicate`. Of course, this situation may not exist in reality. If `MaskAll` and `Replicate` are not supported, then `VectorLongToMask` should not be supported either, and this function will not be called.
>> 
>> My suggestion was to check for Op_Replicate here as Op_MaskAll is already checked underneath VectorNode::scalar2vector under an assumption that MaskAll is a special case for replicate applicable to masks
>
> Oh I misunderstood what you meant, now I understand, thank you!

Done, thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25793#discussion_r2227253339


More information about the hotspot-compiler-dev mailing list