RFR: 8322018: Test java/lang/String/CompactString/MaxSizeUTF16String.java fails with -Xcomp
Jaikiran Pai
jpai at openjdk.org
Thu Dec 14 10:05:38 UTC 2023
On Wed, 13 Dec 2023 21:38:43 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
> The test java/lang/String/CompactString/MaxSizeUTF16String.java fails when run with -Xcomp.
>
> Both the java implementation and the intrinsic for StringUTF16.toBytes() allocate memory for a copy of the string.
> The java implementation of `toBytes()` throws an exception with a message in terms of length of the string.
> The intrinsic uses a generic message when allocating a byte array that is too large for the implementation.
>
> Test should accept either message on the OOME exception, the message is an implementation detail and should reflect the cause of the error and not be confused with a general out of java heap message.
The update to the definition introduces a new `@run` with `-Xcomp` so as to explicitly control the test run instead of relying on external test launch mechanisms to pass around the `-Xcomp`. That then means that the introduction of `@requires vm.flagless`, in this change, is fine.
The change in the test logic itself looks reasonable to me based on what's explained in the PR description.
-------------
Marked as reviewed by jpai (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/17095#pullrequestreview-1781459943
More information about the core-libs-dev
mailing list