RFR: 8214363: HeapWord should not be a fake class
Thomas Schatzl
thomas.schatzl at oracle.com
Fri Feb 15 12:58:27 UTC 2019
Hi Kim,
On Thu, 2019-02-14 at 17:53 -0500, 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/
>
> Testing:
> mach5 tier1-3.
> Built using gcc8.2 and verified there is no longer a -Wclass-
> memaccess warning from objectStartArray.cpp.
looks good. Thanks.
Thomas
More information about the hotspot-dev
mailing list