RFR: 8335392: C2 MergeStores: enhanced pointer parsing [v5]
Dean Long
dlong at openjdk.org
Wed Oct 23 09:08:13 UTC 2024
On Tue, 22 Oct 2024 07:06:11 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> It is not necessary, and I actually test that in my gtest. The subtraction below handles the `min_jint` case.
>
> `ASSERT_TRUE(NoOverflowInt(min_jint).abs().is_NaN());`
>
> But if you insist on it I can add it in.
No, it's fine. I missed the fact that the subtract already handles it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19970#discussion_r1812270307
More information about the hotspot-compiler-dev
mailing list