RFR: 8361842: Move input validation checks to Java for java.lang.StringCoding intrinsics [v3]
Volkan Yazici
vyazici at openjdk.org
Mon Jul 21 12:41:45 UTC 2025
On Wed, 16 Jul 2025 05:28:55 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
>> Volkan Yazici 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 12 additional commits since the last revision:
>>
>> - Merge remote-tracking branch 'upstream/master' into strIntrinCheck
>> - Apply review feedback (styling changes)
>> - Minimize the number of touched lines in `vmIntrinsics.hpp`
>> - Remove Markdown-styling in comments
>> - Improve wording of the `VerifyIntrinsicChecks` flag
>> - Merge remote-tracking branch 'upstream/master' into strIntrinCheck
>> - Fix `EUC_JP.java.template` broken due to `encodeASCII` rename
>> - Remove `StringCodingCountPositives`, `String{En,De}code` already cover our cases
>>
>> This reverts commit 196fc5d406851b8e7070c97ac53ca59c4615aad9.
>> - Improve intrinsics in `StringCoding`
>> - Add `StringCodingCountPositives` benchmark
>> - ... and 2 more: https://git.openjdk.org/jdk/compare/c4b2ddec...85f19864
>
> src/hotspot/share/opto/c2_globals.hpp line 665:
>
>> 663: "prints attempted and successful inlining of intrinsics") \
>> 664: \
>> 665: develop(bool, VerifyIntrinsicChecks, false, \
>
> We should add testing that uses this new flag. Maybe we could add a run to the tests that check the affected intrinsics? We could also add it to our (Oracle internal) stress test jobs at higher tiers.
Given @TobiHartmann's later approval, I'm marking this as resolved.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25998#discussion_r2219089077
More information about the core-libs-dev
mailing list