RFR: 8306302: C2 Superword fix: use VectorMaskCmp and VectorBlend instead of CMoveVF/D

Fri May 19 17:17:51 UTC 2023

On Fri, 19 May 2023 06:20:30 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>>> @fg1417 Since I'm basically implementing your suggestion: do you agree with this fix?
>> 
>> Hi @eme64 , nice rewrite!
>> 
>> BTW, have you tested your patch with `-XX:+UseCMoveUnconditionally` and `-XX:+UseVectorCmov` for all jtreg? Thanks.
>
> Thanks @fg1417 for the review!
> 
> Yes, the testing passes up to at least tier5 and stress testing. With and without `-XX:+UseCMoveUnconditionally` and `-XX:+UseVectorCmov`
> 
> Yes, I hope that someone from intel / x86 specialists can review this too :)
> These are candidates: @jatin-bhateja  @sviswa7  @merykitty

@eme64 I will take a look at it next week.

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

PR Comment: https://git.openjdk.org/jdk/pull/13493#issuecomment-1554992957