RFR: 8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
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.
> 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