RFR(XS): 8140239: Fix product build after "8132168: Support IdealGraphVisualizer in optimized build"

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Wed Oct 21 13:44:49 UTC 2015


Goetz, thanks for spotting that!

Why not simply exclude IGVPrintLevel in product binaries instead?

-    cflags(IGVPrintLevel,           intx, PrintIdealGraphLevel, 
IGVPrintLevel) \
+    NOT_PRODUCT(cflags(IGVPrintLevel, intx, PrintIdealGraphLevel, 
IGVPrintLevel)) \

IGVPrintLevel is used only in non-product code and PrintIdealGraphLevel 
functionality is not available in product builds.

Best regards,
Vladimir Ivanov

PS: I'm surprised the problem wasn't caught by JPRT. Shouldn't the 
merged version be tested? It looks like the merge [1] happened after the 
job finished. (FTR I had to restart the job due to a timeout.)

[1] http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/03fa0a35a468

On 10/21/15 3:50 PM, Lindenmaier, Goetz wrote:
> Hi,
>
> After "8132168: Support IdealGraphVisualizer in optimized build" the
> product build is broken:
> compilerDirectives.cpp:181: error: 'PrintIdealGraphLevel' was not
> declared in this scope
>
> 8132168 changes the flag from 'develop' to 'notproduct'. 'Notproduct'
> does not define
> "const intx PrintIdealGraphLevel = 0;" in the product build, as
> 'develop' did.
>
> I fixed this by wrapping the flag with NOT_PRODUCT:
>
> http://cr.openjdk.java.net/~goetz/webrevs/8140239-prodBld/webrev.00/
>
> Please review this change.  I please need a sponsor.
>
> Best regards,
>
>    Goetz.
>


More information about the hotspot-compiler-dev mailing list