RFR: 8375653: C2: CmpUNode::sub is not monotonic [v5]
Quan Anh Mai
qamai at openjdk.org
Fri Jan 23 08:24:32 UTC 2026
On Fri, 23 Jan 2026 07:40:52 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Improve IR tests
>
> test/hotspot/jtreg/compiler/ccp/TestCmpUMonotonicity.java line 35:
>
>> 33: public class TestCmpUMonotonicity {
>> 34: public static void main(String[] args) {
>> 35: for (int i = 0; i < 20000; i++) {
>
> But do you really need 20000 iterations to trigger the issue? We end up executing the inner loop 20000*50 times. But it's not much work in the loop, so probably does not matter too much.
Thanks for your approval, the inner loop update statement is `i *= step` instead of `i += step`, so the inner loop runs only 7 iterations.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29308#discussion_r2720143519
More information about the hotspot-compiler-dev
mailing list