RFR: 8310906: Fix -Wconversion warnings in runtime, oops and some code header files. [v3]

Coleen Phillimore coleenp at openjdk.org
Wed Jun 28 12:37:04 UTC 2023


On Wed, 28 Jun 2023 07:52:35 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Taking out that cast does work, so I've fixed that.
>
>> pointer_delta has different semantics
> 
> Right. That was "recently" added to pointer_delta with JDK-8260046. It begs the question why felt the need to add it there but feel that it is OK to skip it for delta_as_int? Is there some usage of delta_as_int that gives back negative values? Could that call site be changed?

There are sites where the result is negative but this is a good suggestion because it makes the name more consistent.  I can change those to plain check_cast<int>s.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14675#discussion_r1245141370


More information about the hotspot-dev mailing list