RFR: 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp (original) (raw)

Kim Barrett kim.barrett at oracle.com
Thu Mar 1 00:43:35 UTC 2018


On Feb 28, 2018, at 6:50 PM, coleen.phillimore at oracle.com wrote:

This looks good. Coleen

Thanks.

On 2/28/18 6:46 PM, 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/

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.



More information about the hotspot-dev mailing list