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