RFR: 8336466: C2: Parser incorrectly/unnecessarily checks for clinits

Vladimir Kozlov kvn at openjdk.org
Tue Jul 16 16:32:52 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

Good.

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

Marked as reviewed by kvn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20191#pullrequestreview-2180763251


More information about the hotspot-compiler-dev mailing list