RFR: 8192061: Clean up allocation.inline.hpp includes
Erik Österlund
erik.osterlund at oracle.com
Wed Nov 29 13:35:30 UTC 2017
Hi Stefan,
Nice. Noticed though that in sharedPathsMiscInfo.hpp, it seems like you
accidentally (I assume) added an include of allocation.inline.hpp.
http://cr.openjdk.java.net/~stefank/8192061/webrev.01.16.SHARED.growableArray/src/hotspot/share/classfile/sharedPathsMiscInfo.hpp.sdiff.html
Thanks,
/Erik
On 2017-11-29 12:21, Stefan Karlsson wrote:
> Hi all,
>
> Please review this patchset to clean up our includes of
> allocation.inline.hpp and move usages of its functions out from header
> files into .cpp and .inline.hpp files.
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.all
>
> I've split this patch into multiple separate patches and tried to
> group them according to HotSpot area, to make it easier to review. I
> can also create separate bugs for this, if that's preferable.
>
> GC:
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.00.GC.g1ConcurrentRefine/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.01.GC.spaceCounters/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.02.GC.g1ParScanThreadState.hpp/
>
>
> RT:
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.03.GC.adaptivePaddedAverage/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.04.RT.diagnosticArgument/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.05.RT.methodHandles/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.06.RT.constantPool/
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.07.RT.sharedPathsMiscInfo/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.08.RT.generateOopMap/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.09.RT.objectMonitor/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.10.RT.arguments/
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.11.RT.osThread/
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.12.resourceArea/
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.13.RT.array/
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.14.RT.test_logMessage/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.15.RT.park/
>
> SHARED:
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.16.SHARED.growableArray/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.17.SHARED.removeIncludes/
>
> http://cr.openjdk.java.net/~stefank/8192061/webrev.01.18.SHARED.precompiled/
>
>
> Thanks,
> StefanK
More information about the hotspot-dev
mailing list