RFR: 8253899: Make IsClassUnloadingEnabled signature match specification
Vladimir Kempik
vkempik at openjdk.java.net
Fri Oct 2 07:37:03 UTC 2020
On Fri, 2 Oct 2020 07:27:17 GMT, David Holmes <dholmes at openjdk.org> wrote:
> Okay but look at the example that documentation gives:
>
> > For example, if the jvmtiParamInfo returned by GetExtensionEvents indicates that there is a jint parameter, the event
> > handler should be declared: ```
> > void JNICALL myHandler(jvmtiEnv* jvmti_env, jint myInt, ...)
> > ```
>
> The myInt is explicit, just as our "jboolean* enabled" is explicit. I think they key point is that the signature must
> end with "..." which it does.
> I don't see anything here that needs to be fixed.
Hello David. On majority of platforms this would be fine.
But on some platforms, variadic arguments and non variadic arguments are passed differently ( for example on
macos-aarch64, variadic args are passed always on stack, non variadic on registers (and on stack for 9th+ arg) , that
causes issues.
If you still see no issues here we can delay and make this changeset part of JEP-391.
But since this changeset isn't much macos-aarch64 specific, I thought it would be good to integrate it separately from
jep-391.
Regards, Vladimir
-------------
PR: https://git.openjdk.java.net/jdk/pull/466
More information about the serviceability-dev
mailing list