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

Thomas Schatzl thomas.schatzl at oracle.com
Wed Jan 16 10:57:32 UTC 2019


Hi Zhengyu, Sangheon, Kim,

  thanks for your reviews.

Thomas

On Tue, 2019-01-15 at 20:03 -0500, zgu at redhat.com wrote:
> 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