8230664: Fix TestInstanceKlassSize for PowerPC

Doerr, Martin martin.doerr at sap.com
Mon Sep 28 18:11:16 UTC 2020


Hi,

@Jose:
Thanks for finding this issue.

@all:
The problem is that the implementations in the following files don't fit together:

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/InstanceKlass.java:
public static boolean shouldStoreFingerprint()
public boolean hasStoredFingerprint()

src/hotspot/share/oops/instanceKlass.cpp:
bool InstanceKlass::should_store_fingerprint(bool is_hidden_or_anonymous)
bool InstanceKlass::has_stored_fingerprint()

That breaks it on PPC64.

Why do we require a fingerprint when built with "#if INCLUDE_AOT" and Arguments::is_dumping_archive(), but UseAOT is off?

I could live with Jose's proposal, but a consistent implementation would be much better.

Best regards,
Martin


> -----Original Message-----
> From: hotspot-compiler-dev <hotspot-compiler-dev-
> retn at openjdk.java.net> On Behalf Of joserz at linux.ibm.com
> Sent: Montag, 28. September 2020 14:35
> To: hotspot-compiler-dev at openjdk.java.net
> Cc: Langer, Christoph <christoph.langer at sap.com>
> Subject: 8230664: Fix TestInstanceKlassSize for PowerPC
> 
> Hello team!
> 
> This is an attempt to fix bug JDK-8230664. Please, could you review it?
> 
> PR: https://github.com/openjdk/jdk/pull/358
> Bug: https://bugs.openjdk.java.net/browse/JDK-8230664
> 
> Thank you very much,
> 
> Jose R Ziviani


More information about the hotspot-compiler-dev mailing list