RFR: 8306851: Move Method access flags [v3]

Coleen Phillimore coleenp at openjdk.org
Fri Apr 28 15:43:03 UTC 2023


On Fri, 28 Apr 2023 14:18:11 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove bool argument from ConstMethodFlags.set function.
>
> src/hotspot/share/oops/method.hpp line 615:
> 
>> 613:   // has not been computed yet.
>> 614:   bool guaranteed_monitor_matching() const       { return monitor_matching(); }
>> 615:   void set_guaranteed_monitor_matching()         { set_monitor_matching(); }
> 
> Is this method just obsolete now? If so it might be worth replacing the callers with `set_monitor_matching()` unless `set_monitor_matching()` is still meant to be private.

The reason I left that was to anchor the comment.  There is nowhere good to put that in the X macro.  Also, didn't want to fix the callers.  It's a good point about making monitor_matching() private, but also not really doable with the X macro.  So that's why I left it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13654#discussion_r1180548707


More information about the serviceability-dev mailing list