Integrated: 8303804: Fix some errors of If-VectorTest and CMove-VectorTest

Wang Haomin wanghaomin at openjdk.org
Wed Mar 22 07:41:50 UTC 2023


On Wed, 8 Mar 2023 03:52:33 GMT, Wang Haomin <wanghaomin at openjdk.org> wrote:

> After https://bugs.openjdk.org/browse/JDK-8292289 ,  the base class of VectorTestNode changed from Node to CmpNode. So I add two match rule into ad file.
> 
> match(If cop (VectorTest op1 op2));
> match(Set dst (CMoveI (Binary cop (VectorTest op1 op2)) (Binary src1 src2)));
> 
> First error, rule1 shouldn't generate the statement "node->_bottom_type = _leaf->bottom_type();".
> Second error, both rule1 and rule2 need to use VectorTestNode, the VectorTestNode should be cloned like CmpNode.

This pull request has now been integrated.

Changeset: c039d266
Author:    Wang Haomin <wanghaomin at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c039d26603e85ae37b0a53430a47f5751bf911af
Stats:     4 lines in 2 files changed: 2 ins; 0 del; 2 mod

8303804: Fix some errors of If-VectorTest and CMove-VectorTest

Reviewed-by: qamai, thartmann

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

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


More information about the hotspot-compiler-dev mailing list