RFR: 8214363: HeapWord should not be a fake class

Per Liden per.liden at oracle.com
Mon Feb 18 11:56:22 UTC 2019


On 02/14/2019 11:53 PM, Kim Barrett wrote:
> Please review this change to the definitions of HeapWord and MetaWord.
> They are no longer a class type.  Rather, they are typedefs for a
> pointer to associated incomplete (and never defined) helper classes.
> Thus, we no longer pretend to have class objects where they never
> existed.
> 
> Among other things, this makes memset on a HeapWord[] no longer trigger
> gcc8's new -Wclass-memaccess in ObjectStartArray::set_covered_region().
> 
> Other than the change to the definitions of HeapWord and MetaWord,
> most of the changes are removal of no longer valid forward class
> declarations for those types.
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8214363
> 
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8214363/open.02/

Looks good!

/Per

> 
> Testing:
> mach5 tier1-3.
> Built using gcc8.2 and verified there is no longer a -Wclass-memaccess
> warning from objectStartArray.cpp.
> 


More information about the hotspot-dev mailing list