RFR: 8276711: compiler/codecache/cli tests failing when SegmentedCodeCache used with -Xint [v2]

Coleen Phillimore coleenp at openjdk.java.net
Wed Mar 2 13:26:00 UTC 2022


On Wed, 2 Mar 2022 02:44:18 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Move check for -Xint.
>
> src/hotspot/share/compiler/compilerDefinitions.cpp line 526:
> 
>> 524:   }
>> 525: 
>> 526:   // TieredStopAtLevel==0 allocates nmethod space in the code heap with
> 
> I would prefer to not have special case `TieredStopAtLevel==0`. Unless you want to test loom when JIT compilers are disabled. Currently we don't run any tests with such setting.

I tested with this setting and it works fine with -XX:+SegmentedCodeCache because the code for segmented code cache only checks Arguments::is_interpreter_only().  This doesn't seem right to me, but the testing also expects this to work.  I could fix the tests to not do this. Now I don't know what anyone wants!

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

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


More information about the hotspot-dev mailing list