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