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