RFR (XS) CR 8006403: Regression: jstack failed due to the FieldInfo regression in SA

[Vladimir Kozlov]

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