RFR (S): 8140424: don't prefix developer and notproduct flag variables with CONST_ in product builds
goetz.lindenmaier at sap.com
Fri Nov 6 11:38:49 UTC 2015
thanks for doing this change. It looks good.
It's good that now notproduct flags are also defined in the product build as constants,
as it's already the case with debug flags.
This allows more cleanups in the style you did to compilerDirectives.hpp.
I had a look at these cleanups, quite a row you can find in this patch:
Does it make sense to commit these along with your change?
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On
> Behalf Of Christian Thalinger
> Sent: Freitag, 6. November 2015 07:23
> To: hotspot-dev at openjdk.java.net
> Subject: RFR (S): 8140424: don't prefix developer and notproduct flag
> variables with CONST_ in product builds
> When working on JDK-8024545 I added code that prefixes developer and
> notproduct flags with CONST_. I think I did this to be able to access the
> address of these variables for the flag table. But this is not necessary because
> these flag cannot be written anyway.
> I am proposing to remove the CONST_ prefix and define the variables as
More information about the hotspot-dev