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

Emanuel Peter epeter at openjdk.org
Fri May 19 06:22:49 UTC 2023


On Thu, 11 May 2023 03:56:42 GMT, Fei Gao <fgao at openjdk.org> wrote:

>> @fg1417 Since I'm basically implementing your suggestion: do you agree with this fix?
>
>> @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 :)

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

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


More information about the hotspot-compiler-dev mailing list