On Thu, 12 Mar 2026 08:34:51 GMT, Volkan Yazici <[email protected]> wrote:
>> Improve input validation checks and documentation of the >> `java.lang.StringUTF16` intrinsics. >> >> ### References: >> >> - [JDK-8374582] #29164 — ditto for `java.lang.StringCoding` >> - [JDK-8367129] #28832 — ditto for `java.lang.StringLatin1` >> >> [JDK-8367129]: https://bugs.openjdk.org/browse/JDK-8367129 >> [JDK-8374582]: https://bugs.openjdk.org/browse/JDK-8374582 > > 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 nine additional > commits since the last revision: > > - Merge remote-tracking branch 'upstream/master' into sU16 > - Replace `RETURN_ON_BAILOUT` with two methods > - Move `if (stopped())` into `while (0)` > - `with_opaque` of `generate_string_range_check` become redundant, remove it > - Sharpen class visibility > - Guard `getChars()` > - Guard `toBytes()` > - Factor bailing logic in `generate_string_range_check` out > - Document `getChar`/`putChar` src/hotspot/share/opto/library_call.cpp line 1143: > 1141: if (too_many_traps(Deoptimization::Reason_intrinsic)) { > 1142: return false; > 1143: } @dafedafe, can this be removed too? src/hotspot/share/opto/library_call.cpp line 1288: > 1286: if (too_many_traps(Deoptimization::Reason_intrinsic)) { > 1287: return false; > 1288: } @dafedafe, do we still need this too? src/hotspot/share/opto/library_call.cpp line 1391: > 1389: if (too_many_traps(Deoptimization::Reason_intrinsic)) { > 1390: return false; > 1391: } @dafedafe, this one? src/hotspot/share/opto/library_call.cpp line 1464: > 1462: if (too_many_traps(Deoptimization::Reason_intrinsic)) { > 1463: return false; > 1464: } @dafedafe, what about this one? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30139#discussion_r2932808359 PR Review Comment: https://git.openjdk.org/jdk/pull/30139#discussion_r2932815436 PR Review Comment: https://git.openjdk.org/jdk/pull/30139#discussion_r2932824677 PR Review Comment: https://git.openjdk.org/jdk/pull/30139#discussion_r2932827542
