Integrated: 8294468: Fix char-subscripts warnings in Hotspot
Aleksey Shipilev
shade at openjdk.org
Wed Oct 19 18:54:21 UTC 2022
On Tue, 27 Sep 2022 17:27:03 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> There seem to be the only place in Hotspot where this warning fires, yet the warning is disabled wholesale for Hotspot. This is not good.
>
> I can trace the addition of char-subscripts exclusion to [JDK-8211029](https://bugs.openjdk.org/browse/JDK-8211029) (Sep 2018). The only place in Hotspot where in fires is present from the initial load (2007).
>
> The underlying problem that this warning tells us about is that `char` might be signed on some platforms, so we can potentially access the negative index. It is not a bug in our current code, that bounds the value of `k` under `MAXID-1`, which is `19`.
>
> Additional testing:
> - [x] Linux x86_64 fastdebug `tier1`
> - [x] The build matrix of:
> - GCC 10
> - {i686, x86_64, aarch64, powerpc64le, s390x, armhf, riscv64}
> - {server}
> - {release, fastdebug}
This pull request has now been integrated.
Changeset: ceb5b089
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/ceb5b08964e34dfae3819257e5df460f24f92a78
Stats: 4 lines in 2 files changed: 1 ins; 2 del; 1 mod
8294468: Fix char-subscripts warnings in Hotspot
Reviewed-by: dholmes, kbarrett
-------------
PR: https://git.openjdk.org/jdk/pull/10455
More information about the build-dev
mailing list