RFR: 8224201: Simplify JVM flag macro expansions

Thomas Schatzl thomas.schatzl at oracle.com
Tue May 21 10:41:40 UTC 2019


Hi,

On Mon, 2019-05-20 at 12:56 +0200, Stefan Karlsson wrote:
> Hi all,
> 
> Please review this patch to simplify how we generate the different
> parts of the JVM flags code.
> 
> https://cr.openjdk.java.net/~stefank/8224201/webrev.01/
> https://bugs.openjdk.java.net/browse/JDK-8224201
> 
> Notable parts of the patch:
> - Introduces a ALL_FLAGS macro, used to expand code for all flags.
> - Uses the ALL_FLAGS where appropriate.
> - Constraints list didn't expand JVMCI flags. Fixed by using
> ALL_FLAGS.
> - Moved <sub-system>_globals.hpp generated declarations to be
> expanded 
> in globals.hpp.
> - Unified FLAG_MEMBER and FLAG_MEMBER_WITH_TYPE into
> FLAG_MEMBER_ENUM 
> and change names accordingly.
> 
> This change makes it easier to do future changes to the JVM flags 
> handling code. For example: "JDK-8224203Remove need to specify type
> when using FLAG_SET macros"

  although I saw that you already have quite a few reviewers after
working through it: looks good. Very nice cleanup.

Thanks,
  Thomas




More information about the hotspot-dev mailing list