RFR: 8027229: Lambda: ICCE for 2 or more maximally specific methods for interfaces

Karen Kinnear karen.kinnear at oracle.com
Tue Nov 12 18:34:44 PST 2013


Many thanks. Applied both fixes.

thanks,
Karen

On Nov 12, 2013, at 5:17 PM, John Rose wrote:

> Reviewed.  This is an improvement.  Good comments and trace output, and the logic is cleaner.  — Joh
> 
> P.S.  There's a stray newline in this change:
> +    // the search could find a miranda or a default method
> +    if (is_miranda(m, ik()->methods(), ik()->default_methods(), ik()->super()))
> +{
> +      return true;
> +    }
> 
> ...and also a stray space here before the arrow:
> +  if (original_methods ->length() >  0) {
> +    MetadataFactory::free_array(cld, original_methods);
> +  }
> 
> 
> On Oct 31, 2013, at 10:53 AM, Karen Kinnear <karen.kinnear at oracle.com> wrote:
> 
>> webrev: http://cr.openjdk.java.net/~acorn/8027229/webrev/
>> bug: http://bugs.openjdk.java.net/browse/JDK-8027229
>> 
>> Added support for default method inheritance logic for interfaces.
>> Removed interface methods from interface vtables.
>> Better cleanup of 8027304 as well.
> 



More information about the hotspot-dev mailing list