RFR(XS): 8140239: Fix product build after "8132168: Support IdealGraphVisualizer in optimized build"
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Wed Oct 21 14:34:22 UTC 2015
Hi Vladimir,
I just tried to implement the same behavior as with 'develop'.
But excluding the flag altogether is fine with me, too:
http://cr.openjdk.java.net/~goetz/webrevs/8140239-prodBld/webrev.01/
I put in you as reviewer, if that's ok.
Best regards,
Goetz.
> -----Original Message-----
> From: Vladimir Ivanov [mailto:vladimir.x.ivanov at oracle.com]
> Sent: Mittwoch, 21. Oktober 2015 15:45
> To: Lindenmaier, Goetz; hotspot compiler
> Subject: Re: RFR(XS): 8140239: Fix product build after "8132168: Support
> IdealGraphVisualizer in optimized build"
>
> 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