On Fri, 30 Aug 2024 05:11:30 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> David Holmes has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Exclude test on 32-bit > > test/hotspot/jtreg/runtime/jni/checked/TestLargeUTF8Length.java line 27: > >> 25: * @bug 8328877 >> 26: * @summary Test warning for GetStringUTFLength and functionality of >> GetStringUTFLengthAsLong >> 27: * @library /test/lib > > Shouldn't this test have: > > `@requires vm.bits == 64 > ` Thanks for taking a look @plummercj . Yep I suppose it should. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20784#discussion_r1737947827