hg: jdk7/hotspot-comp/hotspot: 6973963: SEGV in ciBlock::start_bci() with EA

[Vladimir Kozlov]

You mean casting to uintptr_t?
+ _allocation = (uintptr_t)badHeapOopVal;

Yes, please.

Thanks,
Vladimir

Christian Thalinger wrote:
> On Wed, 2010-08-04 at 00:56 +0000, vladimir.kozlov at oracle.com wrote:
>> Changeset: 0e35fa8ebccd
>> Author:    kvn
>> Date:      2010-08-03 15:55 -0700
>> URL:       http://hg.openjdk.java.net/jdk7/hotspot-comp/hotspot/rev/0e35fa8ebccd
>>
>> 6973963: SEGV in ciBlock::start_bci() with EA
>> Summary: Added more checks into ResourceObj and growableArray to verify correctness of allocation type.
>> Reviewed-by: never, coleenp, dholmes
> 
> I get a compiler warning with GCC 4.1.2 after this change:
> 
> src/share/vm/memory/allocation.cpp: In static member function ‘static void ResourceObj::operator delete(void*)’:
> src/share/vm/memory/allocation.cpp:61: warning: negative integer implicitly converted to unsigned type
> src/share/vm/memory/allocation.cpp: In destructor ‘ResourceObj::~ResourceObj()’:
> src/share/vm/memory/allocation.cpp:107: warning: negative integer implicitly converted to unsigned type
> 
> Should I fix it in one of my changes?
> 
> -- Christian
>