On Thu, 23 Feb 2023 16:21:53 GMT, Roger Riggs <rri...@openjdk.org> wrote:
>> Add an `indexOf()` variant allowing to specify both a lower and an upper >> bound on the search. > > Is there any performance impact? Are there relevant JHM tests? Or should be > added. > Also to verify that the intrinsics are still valid though the parameter names > have changed. @RogerRiggs Which performance impacts are you concerned with? Perhaps the additional invocation detour in `indexOf(int ch, int fromIndex)`? AFAICT, there's nothing in `vmIntrinsic.[hpp|cpp]` that would relate to the Java parameter names. Only the method name and the (encoded) parameter types appear there. But of course, it would cost nothing to change them back if I'm wrong. ------------- PR: https://git.openjdk.org/jdk/pull/12600
