RFR: 8302590: Add String.indexOf(int ch, int fromIndex, int toIndex)
Raffaello Giulietti
rgiulietti at openjdk.org
Thu Feb 23 16:50:06 UTC 2023
On Thu, 23 Feb 2023 16:21:53 GMT, Roger Riggs <rriggs at 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
More information about the core-libs-dev
mailing list