Review request for JDK-8008658: Four new method param jtreg tests fail in nightly tests
Jonathan Gibbons
jonathan.gibbons at oracle.com
Thu Feb 21 11:22:10 PST 2013
Looks good to me.
-- Jon
On 02/21/2013 11:21 AM, Eric McCorkle wrote:
> Done.
>
> On 02/21/13 13:38, Jonathan Gibbons wrote:
>> Approved if you add the bug number to the relevant tests.
>>
>> -- Jon
>>
>> On 02/21/2013 10:12 AM, Eric McCorkle wrote:
>>> Hello,
>>>
>>> Please review this patch, which corrects jtreg failures. The root cause
>>> of this failure is a bit complicated... It the result of an improper
>>> merge of two patches.
>>>
>>> The following patch implemented the new format in javac:
>>> http://cr.openjdk.java.net/~emc/8006948/webrev.01/src/share/classes/com/sun/tools/javac/jvm/ClassWriter.java.sdiff.html
>>>
>>>
>>> Note that it converts two appendInt calls to appendChar.
>>>
>>> I had previously committed this patch:
>>> http://cr.openjdk.java.net/~emc/8006345/webrev.04/src/share/classes/com/sun/tools/javac/jvm/ClassWriter.java.sdiff.html,
>>>
>>> which added the second loop.
>>>
>>> A fresh checkout reveals that only the second appendInt got converted to
>>> appendChar. The first appears to have been improperly merged. This
>>> obviously causes invalid MethodParameter attributes to be generated.
>>>
>>> This patch corrects the problem by converting the first appendInt, as it
>>> should have been in 8006948.
>>>
>>> The webrev is here:
>>> http://cr.openjdk.java.net/~emc/8008658/
>>>
>>> Thanks,
>>> Eric
More information about the compiler-dev
mailing list