RFR: 8320530: has_resolved_ref_index flag not restored after resetting entry [v2]

David Holmes dholmes at openjdk.org
Tue Dec 5 10:29:46 UTC 2023


On Thu, 30 Nov 2023 13:39:36 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I see, I said the opposite.  I don't like the #ifdef ASSERT blocks.  3 lines vs 1.
>
> I think each variable should have DEBUG_ONLY() around it, not grouped though.

> 3 lines vs 1

Note I said multi-line DEBUG_ONLY. Single lines should use DEBUG_ONLY

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16769#discussion_r1415315978


More information about the hotspot-dev mailing list