On Wed, 17 Jul 2024 17:15:32 GMT, Andrew Haley <a...@openjdk.org> wrote:

>> src/hotspot/share/oops/klass.inline.hpp line 122:
>> 
>>> 120:     return true;
>>> 121: 
>>> 122:   bool result = lookup_secondary_supers_table(k);
>> 
>> Should `UseSecondarySupersTable` affect `Klass::search_secondary_supers` as 
>> well?
>
> I think not. It'd complicate C++ runtime for no useful reason.

On the other hand, if `-XX:-UseSecondarySupersTable` is intended solely for 
diagnostic purposes, then handling all possible execution modes uniformly is 
preferable, since it gives more confidence when troubleshooting seemingly 
related failures.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19989#discussion_r1683419259

Reply via email to