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