Integrated: 8336466: C2: Parser incorrectly/unnecessarily checks for clinits
Aleksey Shipilev
shade at openjdk.org
Fri Jul 19 08:37:40 UTC 2024
On Tue, 16 Jul 2024 10:12:35 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> As shown in [JDK-8336103](https://bugs.openjdk.org/browse/JDK-8336103), C2 Parser code uses `is_initializer` in the code where we expect to only check for constructors. The fix nominally changes the behavior for C2. I do not believe this is something readily observable, but need to be more explicit.
>
> Additional testing:
> - [x] Linux AArch64 server fastdebug, all (includes Fuzzer and CTW tests), passed as part of JDK-8336103 testing
This pull request has now been integrated.
Changeset: 5d965f36
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/5d965f36d3cd7a1a1f22bf119a98a9e9576a647f
Stats: 5 lines in 2 files changed: 0 ins; 2 del; 3 mod
8336466: C2: Parser incorrectly/unnecessarily checks for clinits
Reviewed-by: kvn, thartmann
-------------
PR: https://git.openjdk.org/jdk/pull/20191
More information about the hotspot-compiler-dev
mailing list