RFR: 8345580: Remove const from Node::_idx which is modified [v2]
Yagmur Eren
duke at openjdk.org
Thu Jan 9 08:55:41 UTC 2025
On Wed, 8 Jan 2025 14:51:34 GMT, Yagmur Eren <duke at openjdk.org> wrote:
>> Ok, just discussed it with some other engineers:
>> Please file a Follow-up RFE. Make `Node::_idx` private, and replace all uses with an accessor.
>
>> You need 2 people with the reviewer role for this, not sure if it is the intention of @eme64
>
> Yes, I realized a bit late that the second one was a committer approval... Thanks for highlighting it @merykitty.
> @nelanbu do you want to work on https://bugs.openjdk.org/browse/JDK-8347275 yourself, or should I find a new-hire / beginner external to work on this?
I have seen that @merykitty took over it. Thanks for taking care of it!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22646#issuecomment-2579486284
More information about the hotspot-compiler-dev
mailing list