RFR: 8315066: Add unsigned bounds and known bits to TypeInt/Long [v15]
Quan Anh Mai
qamai at openjdk.org
Tue Sep 10 18:03:45 UTC 2024
On Tue, 10 Sep 2024 16:56:25 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> Yes it is, but I don't want to remove it. I think it should be the case the `new_type` contains `old_type`, but apparently sometimes it is not the case and I don't understand it. Returning the bottom type instead of `limit_type` is also weird, too.
>
> Unless you want to investigate it this should be removed. File RFE to look on it later.
> Also you can execute it only in debug VM by using #ifdef ASSERT and `assert()` instead of `fatal()`.
Done! I don't want to risk breaking anything so I will take a look later.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17508#discussion_r1752437178
More information about the hotspot-compiler-dev
mailing list