On Tue, 4 Mar 2025 13:35:50 GMT, Jorn Vernee <jver...@openjdk.org> wrote:
>> I reviewed the other use of `intrinsicData`, `tableSwitch`. I believe the >> intrinsic is actually a regression by growing the bytecode size - we should >> just select a MH via hash table lookup and invoke that MH, given all MHs in >> that list have the same type. I have removed the use of intrinsic data here >> and we can move on to remove it later. >> >> I noticed that intrinsics are useful really only as part of named functions. >> And named functions only reuse arbitrary MHs for the invoker form. Is my >> understanding here correct? > >> we should just select a MH via hash table lookup and invoke that MH > > I had something like this in an early prototype of the `tableSwitch` > combinator, but it does not work, as it prevents the method handle calls for > each case from being inlined. FYI this is being addressed in #23763 ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23706#discussion_r1979539450
