RFR(s): 8197992: Move the OopStorage::ParState type out of inline.hpp
Per Liden
per.liden at oracle.com
Thu Feb 15 12:29:40 UTC 2018
Thanks Thomas!
/Per
On 02/15/2018 01:28 PM, Thomas Schatzl wrote:
> Hi Per,
>
> On Thu, 2018-02-15 at 10:31 +0100, Per Liden wrote:
>> Please view this patch, which moves the OopStorage::ParState type
>> out
>> from oopStorageParState.inline.hpp and into a new
>> oopStorageParState.hpp. The functions depending on other .inline.hpp
>> includes remain in oopStorageParState.inline.hpp.
>>
>> This patch is needed to allow classes defined in other .hpp files to
>> contain a oopStorage::ParState instance by value (without having
>> inline.hpp includes in .hpp file). ZGC's ZRootIterator is an example
>> of
>> this.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8197992
>> Webrev: http://cr.openjdk.java.net/~pliden/8197992/webrev.0/
>> Testing: hs-tier1
>
> the refactoring looks good to me.
>
> Thomas
>
More information about the hotspot-gc-dev
mailing list