RFR: 8303804: Fix some errors of If-VectorTest and CMove-VectorTest [v2]
Wang Haomin
wanghaomin at openjdk.org
Thu Mar 9 01:19:40 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.
Wang Haomin has updated the pull request incrementally with one additional commit since the last revision:
compare the results with 0
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/12917/files
- new: https://git.openjdk.org/jdk/pull/12917/files/bfba75dd..564d0c48
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=12917&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=12917&range=00-01
Stats: 2 lines in 1 file changed: 0 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