RFR: 8292898: [vectorapi] Unify vector mask cast operation [v4]

Tue Sep 27 03:32:20 UTC 2022

On Tue, 27 Sep 2022 03:25:47 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> `VecMaskUseAll` is fine to me since the final vector box may need to store mask if it is mask casting/reinterpreting. So since it has checked the load/store mask on Line#2439, do you think it needs to double check the load mask pattern here?
>
> Since checks related to VecMaskUseAll are decoupled from the main opcode so one check per intrinsic routine should be ok if its covering the needed trajectories. Infact we are passing VecMaskUseAll back to back to two rountines on Line #2439 and Line # 2452, WDYT ?

Yes, I'm agree that one check is enough.

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

PR: https://git.openjdk.org/jdk/pull/10192