RFR (XS): 8217203: Some more includes to .inline.hpp files in gc header files

zgu at redhat.com zgu at redhat.com
Wed Jan 16 01:03:16 UTC 2019


Looks good to me.

Thanks for fixing ShenandoahTaskqueue.hpp and filing JDK-8217213. I
will handle JDK-8217213.

-Zhengyu


On Wed, 2019-01-16 at 00:13 +0100, Thomas Schatzl wrote:
> Hi all,
> 
>   over time some gc hpp files started directly including .inline.hpp
> files again.
> 
> Fix this to avoid future dependency messes.
> 
> I fixed one of the problems in shenandoah files because it was easy -
> the one in shenandoahTaskqueue.hpp has been split into an extra CR (
> https://bugs.openjdk.java.net/browse/JDK-8217213) as it started
> getting
> complicated fixing wrong includes there.
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8217203
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8217203/webrev/
> Testing:
> local compilation, building in CI
> 
> Thanks,
>   Thomas
> 



More information about the hotspot-gc-dev mailing list