RFR(S): 8026735 Stream tests throw java.lang.IncompatibleClassChangeError

Christian Thalinger christian.thalinger at oracle.com
Wed Oct 30 21:43:15 PDT 2013


Looks good.  Thank you for fixing this.

On Oct 30, 2013, at 7:24 PM, Igor Veresov <igor.veresov at oracle.com> wrote:

> All right. Moved it: http://cr.openjdk.java.net/~iveresov/8026735/webrev.2/
> 
> igor
> 
> On Oct 30, 2013, at 6:32 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
>> On 10/30/13 5:18 PM, Igor Veresov wrote:
>>> Thanks!
>>> 
>>> Here's the renamed updated webrev:
>>> http://cr.openjdk.java.net/~iveresov/8026735/webrev.1/
>> 
>> Don't use bugid 8026735 for subdir. I suggested 'inlining' to collect all jtreg tests related to inlining.
>> 
>> Thanks,
>> Vladimir
>> 
>>> 
>>> igor
>>> 
>>> On Oct 30, 2013, at 4:23 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>> 
>>>> Fix looks good.
>>>> We use new naming for tests now, please, change to next (or better name if you find):
>>>> 
>>>> test/compiler/inlining/InlineDefaultMethod.java
>>>> 
>>>> Thanks,
>>>> Vladimir
>>>> 
>>>> On 10/30/13 3:38 PM, Igor Veresov wrote:
>>>>> CHA in C1 incorrectly treats calls to interfaces with default methods.
>>>>> We decided to put a temporary bandaid to disable CHA-driven inlining for invokeinterface on interfaces with default methods in C1.
>>>>> 
>>>>> Webrev: http://cr.openjdk.java.net/~iveresov/8026735/webrev.0/
>>>>> Testing: affected tests in java/util/stream, the regression test
>>>>> 
>>> 
> 



More information about the hotspot-compiler-dev mailing list