RFR: 8219713: Reduce work in DefaultMethods::generate_default_methods

Claes Redestad claes.redestad at oracle.com
Fri Mar 1 18:55:41 UTC 2019


On 2019-03-01 19:34, Karen Kinnear wrote:
>>
>> http://cr.openjdk.java.net/~redestad/8219713/open.02/
>>
>> What do you think?
>>
>> For reference, the change that added m->is_static() to the examination
>> was: https://bugs.openjdk.java.net/browse/JDK-8028438 Lambda:
>> default methods static superclass method masking default -
> Thank you that is a big help.
> The question was whether the two test cases needed for that fix were 
> ever added to the defmeth tests since
> they were in a separate repo at the time - my memory is they fell 
> through the cracks:
> 
> Test cases added to defmeth StaticMethodsTest in review in parallel. New 
> test cases testStaticSuperClassVsDefaultSuperInterface and 
> testStaticLocalVsDefaultSuperInterface
> 

Seems they came through with the VM test base open sourcing and are both
in test/hotspot/jtreg/vmTestbase/vm/runtime/defmeth/StaticMethodsTest.java

Regardless of where we go with this I think it'd be good with versions
of those tests that solidify the behavior w.r.t. private static method
resolution as well.

Thanks!

/Claes



More information about the hotspot-runtime-dev mailing list