RFR: 8367129: Move input validation checks to Java for java.lang.StringLatin1 intrinsics [v5]
Volkan Yazici
vyazici at openjdk.org
Fri Feb 20 16:29:00 UTC 2026
On Tue, 10 Feb 2026 16:36:18 GMT, Damon Fenacci <dfenacci 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 16 additional commits since the last revision:
>>
>> - Merge remote-tracking branch 'upstream/master' into sLatin1Intr
>> - Remove redundant checks in `indexOf0(byte[] value, byte[] str)`
>> - More `indexOf` et al. guards and other review feedback
>> - Use `vmIntrinsics::*` to refer to intrinsics from Java
>> - Improve vertical alignment in `vmIntrinsics.hpp`
>> - Apply review feedback
>> - More doc updates
>> - Reflect that `VerifyIntrinsicChecks` is removed
>> - Merge remote-tracking branch 'upstream/master' into sLatin1Intr
>> - Fix `Helper`
>> - ... and 6 more: https://git.openjdk.org/jdk/compare/bf17641f...6eb75a06
>
> Thanks for looking into this @vy! I left a couple of comments/questions (I've mostly checked the intrinsic part).
@dafedafe, @RogerRiggs, @rgiulietti, thank you so much for your kind assistance and reviews. I needed some extra time to check the performance implications of the changes (i.e., 6eb75a06d94), but they look good. Tier 1-2 is clear too.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28832#issuecomment-3935864428
More information about the core-libs-dev
mailing list