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