Integrated: 8352248: Check if CMoveX is supported

[Hamlin Li]

On Tue, 18 Mar 2025 10:02:27 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this patch?
> 
> Currenlty, seems CMoveX are fully supported on most platforms, except of riscv64.
> On riscv64, there is no efficient way to implement CMoveF/D as other CMoveX (e.g. CMoveI), but it will still bring benefit by just supporting CMoveX without CMoveF/D. This patch is to supply such option.
> 
> As other platforms already supported CMoveX, this patch should not impact them, as `!CMoveNode::supported(_igvn.type(phi))` should always be false.
> 
> BTW, in a subsequent pr for riscv, I'll implement CMoveX except of CMoveF/D, and also return false for CMoveF/D in Matcher::match_rule_supported.
> 
> Thanks!

This pull request has now been integrated.

Changeset: d1cf2328
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d1cf232893615f1907bb84728cbc1f566a369757
Stats:     20 lines in 3 files changed: 16 ins; 0 del; 4 mod

8352248: Check if CMoveX is supported

Reviewed-by: chagedorn, luhenry, rehn

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

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