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

Wang Haomin wanghaomin at openjdk.org
Wed Mar 8 03:59:51 UTC 2023


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.

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

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

Changes: https://git.openjdk.org/jdk/pull/12917/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12917&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8303804
  Stats: 4 lines in 2 files changed: 2 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/12917.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12917/head:pull/12917

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


More information about the hotspot-compiler-dev mailing list