RFR: 8361842: Move input validation checks to Java for String-related intrinsics [v2]

Volkan Yazici vyazici at openjdk.org
Tue Jul 15 19:21:29 UTC 2025


On Mon, 14 Jul 2025 08:49:44 GMT, Damon Fenacci <dfenacci at openjdk.org> wrote:

>> Volkan Yazici has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - Minimize the number of touched lines in `vmIntrinsics.hpp`
>>  - Remove Markdown-styling in comments
>>  - Improve wording of the `VerifyIntrinsicChecks` flag
>
> src/hotspot/share/opto/c2_globals.hpp line 666:
> 
>> 664:                                                                             \
>> 665:   develop(bool, VerifyIntrinsicChecks, false,                               \
>> 666:           "Verify that Java level checks in intrinsics work as expected")   \
> 
> To make it clearer I think we might want to move "in intrinsic" after "Verify" or at the end.

Changed as suggested in c331fbfa.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25998#discussion_r2208455093


More information about the graal-dev mailing list