RFR(XS): 8211364: Remove expired flags
David Holmes
david.holmes at oracle.com
Tue Oct 2 02:00:18 UTC 2018
Hi Mikael,
On 2/10/2018 11:16 AM, Mikael Vidstedt wrote:
>
> Please review this change which removes expired flags.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8211364
> webrev: http://cr.openjdk.java.net/~mikael/webrevs/8211364/webrev.00/open/webrev/
>
> * Background (from the issue)
>
> A number of flags which expire(d) in jdk12 should be removed from the special_jvm_flags table.
>
> * Testing
>
> tier1-3. Lots of manual grep:ing to make sure none of the flags is actually used.
All of which should have been done when the flags were obsoleted! :(
This all seems fine to me, but obviously the JFR tests were not handled
when the FastTLABRefill flag was deprecated or obsoleted! Someone from
JFR needs to re-examine all those tests to see if they are still testing
what was expected in relation to C1, c2 and interpreter.
Aside: the ObsoleteFlagErrorMessage.java test is annoying in that it
needs updating on each release to find a new "newly obsoleted" flag. :(
Thanks,
David
>
> Note: There’s one reference to InlineNotify (and SyncKnobs) in src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.hotspot/src/org/graalvm/compiler/hotspot/GraalHotSpotVMConfigVersioned.java. I’m following up on that separately.
>
> Cheers,
> Mikael
>
More information about the hotspot-runtime-dev
mailing list