RFR: 8336103: Clean up confusing Method::is_initializer [v4]

Aleksey Shipilev shade at openjdk.org
Sun Oct 6 14:58:42 UTC 2024


On Fri, 27 Sep 2024 09:57:06 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> All around Hotspot, we have calls to `method->is_initializer()`. That method tests for both instance and static initializers. In many cases, the uses imply we actually want to test for constructor, not static initializer. Sometimes we filter explicitly for `!m->is_static()`, sometimes we don't.  At this point, I think the best way to prevent future accidents is to remove the confusing `is_initializer`.
>> 
>> The behavioral changes have been handled by already integrated PRs, see the links in JBS. The changes left here are not (supposed to be) changing the behavior.
>
> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit:
> 
>   Fix

Thank you! I think I need another formal Reviewer comment before I can integrate.

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

PR Comment: https://git.openjdk.org/jdk/pull/20120#issuecomment-2395469026


More information about the graal-dev mailing list