RFR (S) 8228630: Remove always true parameter to NoSafepointVerifier
David Holmes
david.holmes at oracle.com
Wed Jul 31 05:01:12 UTC 2019
Hi Coleen,
On 31/07/2019 6:45 am, coleen.phillimore at oracle.com wrote:
> Summary: Also remove NoGCVerifier since NoSafepointVerifier covers GC
> checking when not already at a safepoint and is a stronger check.
It wasn't at all clear to me that we may not want a NoGCVerifier that is
independent of safepoints e.g. for use in a non-JavaThread. But AFAICS
we don't use NoGCVerifier directly, but only via NoSafepointVerifier -
in which case the "no safepoint" check subsumes the "no gc" check and
the whole thing collapses to what you have (which results in a nice
amount of code deletion!).
It took me a while to follow through all the changes but it seems good.
I spotted this reference to NoGCVerifier:
./share/opto/runtime.cpp:// Thus, it cannot be a leaf since it contains
the NoGCVerifier.
I'm not at all sure that the rest of the comment related to this is
accurate any more - the reference to the NoGCVerifier didn't make sense
to me.
Thanks,
David
-----
> See bug for more details also. Tested with all jtreg
> runtime,compiler,serviceability and gc tests. Also hs-tier1-3 on
> linux-x64-debug.
>
> open webrev at http://cr.openjdk.java.net/~coleenp/2019/8228630.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8228630
> Thanks,
> Coleen
More information about the hotspot-runtime-dev
mailing list