RFR: 8267265: Use new IR Test Framework to create tests for C2 Ideal transformations [v2]

Christian Hagedorn chagedorn at openjdk.java.net
Tue Feb 15 08:42:09 UTC 2022


On Mon, 14 Feb 2022 20:43:12 GMT, Cesar Soares <duke at openjdk.java.net> wrote:

>> test/hotspot/jtreg/compiler/c2/irTests/AddLNodeIdealizationTests.java line 73:
>> 
>>> 71:         Asserts.assertEQ((a - b) + (b + c), test6(a, b, c));
>>> 72:         Asserts.assertEQ((a - b) + (c + b), test7(a, b, c));
>>> 73:         Asserts.assertEQ((a - b) + (c - a), test8(a, b, c));
>> 
>> `AddI` additionally has 
>> 
>> Asserts.assertEQ((a - b) + (b - c), test8(a, b, c));
>> 
>> Was this left out on purpose?
>
> This is another transformation that is missing for "Long". Tracked by this RFE: https://bugs.openjdk.java.net/browse/JDK-8272735

Great! Thanks for the explanation.

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

PR: https://git.openjdk.java.net/jdk/pull/7434


More information about the hotspot-compiler-dev mailing list