RFR (XS) CR 8006403: Regression: jstack failed due to the FieldInfo regression in SA
Jesper Wilhelmsson
jesper.wilhelmsson at oracle.com
Wed Jan 16 09:26:05 PST 2013
OK, since these changes are in hotspot I'll sponsor this change as well.
I'm getting it ready for JPRT now (will do some local testing first) and will
push later today.
/Jesper
On 2013-01-16 18:09, Vladimir Kozlov wrote:
> Good.
>
> Vladimir
>
> On 1/16/13 1:49 AM, Aleksey Shipilev wrote:
>> On 01/16/2013 01:45 PM, Staffan Larsen wrote:
>>> Thanks for jumping on this.
>>
>> That's my sin to write that dumb code in the first place.
>>
>>> Shouldn't this be looked up the same way the other constants are looked up?
>>> Something like:
>>>
>>> FIELDINFO_TAG_MASK = db.lookupIntConstant("FIELDINFO_TAG_MASK").shortValue();
>>
>> Yes, that occurred to me as well, see updated webrev:
>> http://cr.openjdk.java.net/~shade/8006403/webrev.02/
>>
>> Tested that jstack now works fine.
>>
>> -Aleksey.
>>
More information about the hotspot-dev
mailing list