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 10:38:58 PST 2013
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