Re: RFR: 8305093: Linker cache should not take layout names into account [v5]

2023-05-01 Thread Jorn Vernee
> Strip names from the function descriptors used in linkage requests. The names > are not important for classification, and this will allow the cached > MethodHandle/UpcallStubFactory to be shared across linkage request with the > same structure, but with layouts that have different names. > >

Re: RFR: 8305093: Linker cache should not take layout names into account [v4]

2023-04-27 Thread Jorn Vernee
> Strip names from the function descriptors used in linkage requests. The names > are not important for classification, and this will allow the cached > MethodHandle/UpcallStubFactory to be shared across linkage request with the > same structure, but with layouts that have different names. > >

Re: RFR: 8305093: Linker cache should not take layout names into account [v3]

2023-04-27 Thread Jorn Vernee
> Strip names from the function descriptors used in linkage requests. The names > are not important for classification, and this will allow the cached > MethodHandle/UpcallStubFactory to be shared across linkage request with the > same structure, but with layouts that have different names. > >

Re: RFR: 8305093: Linker cache should not take layout names into account [v2]

2023-04-27 Thread ExE Boss
On Fri, 21 Apr 2023 01:03:57 GMT, Jorn Vernee wrote: >> Strip names from the function descriptors used in linkage requests. The >> names are not important for classification, and this will allow the cached >> MethodHandle/UpcallStubFactory to be shared across linkage request with the >> same s

Re: RFR: 8305093: Linker cache should not take layout names into account [v2]

2023-04-27 Thread Andrey Turbanov
On Fri, 21 Apr 2023 01:03:57 GMT, Jorn Vernee wrote: >> Strip names from the function descriptors used in linkage requests. The >> names are not important for classification, and this will allow the cached >> MethodHandle/UpcallStubFactory to be shared across linkage request with the >> same s

Re: RFR: 8305093: Linker cache should not take layout names into account [v2]

2023-04-24 Thread Maurizio Cimadamore
On Fri, 21 Apr 2023 01:03:57 GMT, Jorn Vernee wrote: >> Strip names from the function descriptors used in linkage requests. The >> names are not important for classification, and this will allow the cached >> MethodHandle/UpcallStubFactory to be shared across linkage request with the >> same s

Re: RFR: 8305093: Linker cache should not take layout names into account [v2]

2023-04-24 Thread Jorn Vernee
On Mon, 24 Apr 2023 11:46:57 GMT, Maurizio Cimadamore wrote: >> Jorn Vernee has updated the pull request incrementally with one additional >> commit since the last revision: >> >> use al instead of ValueLayout.ADDRESS > > src/java.base/share/classes/jdk/internal/foreign/abi/AbstractLinker.ja

Re: RFR: 8305093: Linker cache should not take layout names into account [v2]

2023-04-24 Thread Maurizio Cimadamore
On Fri, 21 Apr 2023 01:03:57 GMT, Jorn Vernee wrote: >> Strip names from the function descriptors used in linkage requests. The >> names are not important for classification, and this will allow the cached >> MethodHandle/UpcallStubFactory to be shared across linkage request with the >> same s

Re: RFR: 8305093: Linker cache should not take layout names into account [v2]

2023-04-20 Thread Jorn Vernee
> Strip names from the function descriptors used in linkage requests. The names > are not important for classification, and this will allow the cached > MethodHandle/UpcallStubFactory to be shared across linkage request with the > same structure, but with layouts that have different names. > >

Re: RFR: 8305093: Linker cache should not take layout names into account

2023-04-20 Thread ExE Boss
On Thu, 20 Apr 2023 17:23:08 GMT, Jorn Vernee wrote: > Strip names from the function descriptors used in linkage requests. The names > are not important for classification, and this will allow the cached > MethodHandle/UpcallStubFactory to be shared across linkage request with the > same struc

RFR: 8305093: Linker cache should not take layout names into account

2023-04-20 Thread Jorn Vernee
Strip names from the function descriptors used in linkage requests. The names are not important for classification, and this will allow the cached MethodHandle/UpcallStubFactory to be shared across linkage request with the same structure, but with layouts that have different names. I also have

Re: RFR: 8305093: Linker cache should not take layout names into account

2023-04-20 Thread Jorn Vernee
On Thu, 20 Apr 2023 19:29:40 GMT, ExE Boss wrote: >> Strip names from the function descriptors used in linkage requests. The >> names are not important for classification, and this will allow the cached >> MethodHandle/UpcallStubFactory to be shared across linkage request with the >> same stru