RFR(M) : 7104565 : trim jprt build targets

[Vladimir Kozlov]

On 4/10/13 8:34 AM, David Chase wrote:
> 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?

Keep your ifdef ASSERT. I tried to say that original changes used DEBUG 
instead of ASSERT due to confusion.

Thanks,
Vladimir

>
> 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
>