Integrated: 8355711: Remove incorrect overflow check in RawBytecodeStream::raw_next
Johan Sjölen
jsjolen at openjdk.org
Mon May 5 07:16:53 UTC 2025
On Mon, 28 Apr 2025 13:18:53 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:
> Hi,
>
> This fixes a typical wrong overflow check. `_bci` and `_next_bci` are both `int`, so any overflow is undefined, therefore an optimizing C++ compiler is allowed to remove this check. When looking at the x64 assembly for this, I could not find the check, so I guess that it was actually removed as well.
This pull request has now been integrated.
Changeset: f04e556d
Author: Johan Sjölen <jsjolen at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/f04e556d471d65a9b1a86d33acf72f6ddec944e1
Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
8355711: Remove incorrect overflow check in RawBytecodeStream::raw_next
Reviewed-by: dlong, matsaave
-------------
PR: https://git.openjdk.org/jdk/pull/24920
More information about the hotspot-runtime-dev
mailing list