RFR: 8253002: Remove the unused SafePointNode::_oop_map field [v2]

Jorn Vernee jvernee at openjdk.java.net
Tue Sep 15 09:29:14 UTC 2020


On Mon, 14 Sep 2020 13:19:20 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review comments:
>>   - Remove comment in SafePointNode
>>   - Remove forward declaration of class OopMap
>
> There is also a forward declaration of `class OopMap;` in `callnode.hpp`, do we still need it?

@shipilev Thanks for the review!

> There is also a forward declaration of class OopMap; in callnode.hpp, do we still need it?

Doesn't look like it, good catch!

> I don't think we need this comment. I think it would bitrot eventually. It seems the code to construct OopMap is easily
> discoverable already.

Yeah, good point, I've removed it.

I've ran a build on Windows and Linux (WSL). Still builds with the new changes, I hope that's enough to test them.

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

PR: https://git.openjdk.java.net/jdk/pull/109


More information about the hotspot-compiler-dev mailing list