RFR(XL): 8185640: Thread-local handshakes
Andrew Haley
aph at redhat.com
Mon Oct 23 16:36:03 UTC 2017
This is a bad way to handle supports_thread_local_poll():
static bool supports_thread_local_poll() {
#if defined(AMD64) || defined(SPARC)
return true;
#else
return false;
#endif
}
Instead, it is better to use a flag which is #defined in the back
ends, and allow each back end to specify if it supports thread-local
handshakes. We have *two* AARCH64 back ends, and only one of them
supports thread-local handshakes; both of them #define AARCH64.
#if defined(BLAH) should be reserved for hardware-specific properties,
not back-end-specific properties.
--
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
More information about the hotspot-dev
mailing list