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