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