RFR: 8350642: Interpreter: Upgrade CountBytecodes to 64 bit on 64 bit platforms [v2]

David Linus Briemann duke at openjdk.org
Tue Mar 11 14:44:04 UTC 2025


On Wed, 5 Mar 2025 19:39:23 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> David Linus Briemann has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 15 additional commits since the last revision:
>> 
>>  - remove CountBytecodesTest from tier1
>>  - Merge branch 'master' into dlb/bytecode_counter_overflow
>>  - remove auto included header
>>  - fix x86 asm
>>  - address review comment, add back comma to copyright header
>>  - formatting
>>  - remove bad header
>>  - add missing comma to copyright header
>>  - speed up runtime by running less bytecodes, add explanation
>>  - add copyright header and @bug number
>>  - ... and 5 more: https://git.openjdk.org/jdk/compare/057e808f...31a52156
>
> test/hotspot/jtreg/runtime/interpreter/CountBytecodesTest.java line 32:
> 
>> 30:  *          does not overflow for more than 2^32 bytecodes counted.
>> 31:  * @library /test/lib
>> 32:  * @run main/othervm/timeout=300 CountBytecodesTest
> 
> The long tests should be excluded from tier1. Please update TEST.groups.

I excluded the test from the `tier1_runtime` tests. To my understanding it should now run in tier4. Could you please verify? Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23766#discussion_r1989450498


More information about the hotspot-dev mailing list