RFR(M) : 7104565 : trim jprt build targets
David Chase
david.r.chase at oracle.com
Wed Apr 10 08:34:47 PDT 2013
David, Vladimir (and anyone else)
>> Looking at stackMapFrame.hpp the changeset that added the "ifdef DEBUG" was
>>
>> http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/4ee06e614636
>>
>> and it seems to me that perhaps it should have been
>> "ifndef PRODUCT" based on the fact that it appears to be a refactoring
>> of NOT_PRODUCT code. (The only difference being whether or not it is
>> included in the optimized build.)
>
> I think, the intention of changes was to not include it into optimized build but may be there was confusion about DEBUG vs ASSERT.
Is this a proposed change for this fix in this file (replace ifdef ASSERT with ifndef PRODUCT)
or is this a change for some future fix, or is this a road-not-taken?
I'm happy either way, the editor is open, I just want to get all moles properly whacked.
(other moles -- jvmgminimal1 and llvmHeaders.hpp -- are whacked)
David
More information about the hotspot-dev
mailing list