RFR: 8215708: ZGC: Add missing LoadBarrierNode::size_of()
Erik Österlund
erik.osterlund at oracle.com
Tue Jan 8 14:43:46 UTC 2019
Hi Per,
Looks good.
Thanks,
/Erik
On 2019-01-08 15:32, Per Liden wrote:
> LoadBarrierNode should implement size_of(). Otherwise cloning of such
> nodes is broken since only part of the object will be copied. This
> caused incorrect load barriers to be used in random places. For
> example, we could generate a weak barrier instead of a strong barrier,
> because the _weak member was not properly initialized when cloned.
>
> This patch also implements three other methods (cmp, adr_type and
> match_edge) with an immediate call to ShouldNotReachHere(). This is a
> pure safety net to catch any misuse of these. These should never be
> called, but if they are called today we might not notice and instead
> silently do the wrong thing.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8215708
> Webrev: http://cr.openjdk.java.net/~pliden/8215708/webrev.0
>
> Testing: tier{1,6,7}
>
> /Per
More information about the hotspot-compiler-dev
mailing list