RFR: 8076457: Fix includes of inline.hpp in GC code

Jon Masamitsu jon.masamitsu at oracle.com
Wed Apr 1 19:09:47 UTC 2015

Changes look correct.  Small question.


+// Non-inlined version to be used where thread.inline.hpp shouldn't be included.

Would this be more accurate?

// Non-inlined version to be used where thread.inline.hpp does not need 
to be included

Minor point.  You can decide.  I don'tneed  to see the final comment.


The changes do a bit more than deal with includes of various *.inline.hpp.

Should the description of in the CR be a more general

"Fix more includes in GC code".



On 4/1/2015 4:51 AM, Stefan Karlsson wrote:
> Hi,
> Please review this patch to fix the last few places in the GC code 
> where inline.hpp files are included from hpp files.
> http://cr.openjdk.java.net/~stefank/8076457/webrev.01/
> https://bugs.openjdk.java.net/browse/JDK-8076457
> This patch builds upon the patch in:
> http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2015-April/012607.html 
> Thanks,
> StefanK

More information about the hotspot-gc-dev mailing list