On Tue, 10 Feb 2026 16:36:18 GMT, Damon Fenacci <[email protected]> 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
