RFR: 8297582: C2: very slow compilation due to type system verification code [v3]

Vladimir Kozlov kvn at openjdk.org
Thu Dec 15 20:13:06 UTC 2022


On Thu, 15 Dec 2022 19:48:37 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Vladimir's review
>
> src/hotspot/share/opto/type.cpp line 989:
> 
>> 987:   const Type *mt = this_t->xmeet(t);
>> 988: #ifdef ASSERT
>> 989:   VerifyMeetResult* type_verify = C->_type_verify;
> 
> As a style nit, you could access `VerifyMeetResult*` through `VerifyMeetMark` or even encapsulate it there.

I assume you suggested to hide access  to cache (`add()`, `meet()`) in `VerifyMeetMark`. I agree. `VerifyMeetMark` can be renamed `VerifyMeet` in such case.

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

PR: https://git.openjdk.org/jdk/pull/11673


More information about the hotspot-compiler-dev mailing list