RFR(M) : 7104565 : trim jprt build targets

Stefan Karlsson stefan.karlsson at oracle.com
Thu Apr 4 00:11:57 PDT 2013


David,

You change a lot of make files and this will affect all HotSpot 
developers, so this review should probably go to hotspot-dev and not 
hotspot-compiler-dev.

Have you tried these changes with a full JDK8 forest and the new build 
infrastructure?

thanks,
StefanK

On 04/04/2013 05:00 AM, David Chase wrote:
> webrev: http://cr.openjdk.java.net/~drchase/7104565/webrev.02/
>
> Problem:
> Debug builds are inconsistently named, and are not needed for usual JPRT testing, and consume lots of space and time.
> #ifdef conventions are a little dubious (I'm just paraphrasing from the bug here).
>
> Fix:
> Pick up Tom Rodriguez's work towards a fix (95+% complete),
> translate forward in time to current build, check for omissions and changes.
>
> Fix includes:
> replace #ifdef DEBUG and #ifdef FASTDEBUG with #ifdef ASSERT
> consistently name the debug build "debug" instead of "jvmg"
> remove "debug" targets from JPRT properties files.
>
> Testing:
> Jtreg locally
> JPRT of usual compiler tests
> Cross-checked old diffs with new diffs
> JPRT of compiler regression tests
> Manually check that appropriate -DASSERT flag is passed to compilations.
>
> Noticeable changes:
> "make debug", not "make jvmg"
> JPRT of compiler regression tests is much faster and times out much less often.
>



More information about the hotspot-compiler-dev mailing list