RFR: 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
Per Liden
per.liden at oracle.com
Thu Mar 1 18:28:03 UTC 2018
Hi,
On 2018-03-01 00:46, Kim Barrett wrote:
> Finally, updated webrevs:
> full: http://cr.openjdk.java.net/~kbarrett/8198474/open.01/
> incr: http://cr.openjdk.java.net/~kbarrett/8198474/open.01.inc/
Looks good!
>
> To remove the #include of jniHandles.inline.hpp by
> jvmciCodeInstaller.hpp, I've moved the definitions referring to
> JNIHandles::resolve from the .hpp file to the .cpp file.
>
> For jvmciJavaClasses.hpp, I've left it including
> jniHandles.inline.hpp. It already includes two other .inline.hpp
> files. I'm leaving it to whoever fixes the existing two to fix this
> one as well.
I'd love to see this fixed, but I realize it's non-trivial and would
require quite a bit of work, so I'm ok with leaving this as is for now.
/Per
More information about the hotspot-dev
mailing list