RFR(S): 8143324 - Backout JDK-8087223

[Christian Tornqvist]

The credit for enabling them in nightly should go to Dmitry :)

-----Original Message-----
From: Karen Kinnear [mailto:karen.kinnear at oracle.com] 
Sent: Thursday, November 19, 2015 1:38 PM
To: Christian Tornqvist <christian.tornqvist at oracle.com>
Cc: hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(S): 8143324 - Backout JDK-8087223

Christian,

Thank you for getting the defmeth tests into the nightlies so we caught this
so early and thank you for the backout web rev.

thanks,
Karen

> On Nov 19, 2015, at 1:26 PM, Christian Tornqvist
<christian.tornqvist at oracle.com> wrote:
> 
> Hi Karen,
> 
> The backout was clean and I've manually diffed the files with the file 
> revisions before 8087223 to verify that it cleanly reverts the change.
> 
> The defmeth and the jdk test will have the same result as before 
> 8087223, we did not see any failures in the nightly testing yesterday.
> 
> Thanks,
> Christian
> 
> -----Original Message-----
> From: Karen Kinnear [mailto:karen.kinnear at oracle.com]
> Sent: Thursday, November 19, 2015 12:17 PM
> To: Christian Tornqvist <christian.tornqvist at oracle.com>
> Cc: hotspot-runtime-dev at openjdk.java.net
> Subject: Re: RFR(S): 8143324 - Backout JDK-8087223
> 
> Thank you so much Christian.
> 
> Does this web rev pass the default methods tests and the idk/lambda/vm 
> test that failed?
> 
> thanks,
> Karen
> 
>> On Nov 19, 2015, at 11:33 AM, Christian Tornqvist
> <christian.tornqvist at oracle.com> wrote:
>> 
>> Hi everyone,
>> 
>> 
>> 
>> Please review this change that backs out the changes made in 8087223, 
>> the change caused massive test failures (tracked by JDK-8143320 and 
>> JDK-8143317).
>> 
>> 
>> 
>> Webrev:
>> 
>> http://cr.openjdk.java.net/~ctornqvi/webrev/8143324/webrev.00/
>> 
>> 
>> 
>> Thanks,
>> 
>> Christian
>> 
> 
>