Mercurial > hg > icedtea8-forest > hotspot
changeset 9914:07275619e52e
8190478: Improved interface method selection
Reviewed-by: acorn, ahgross, jwilhelm, rhalade
Contributed-by: harold.seigel@oracle.com
author | hseigel |
---|---|
date | Tue, 05 Dec 2017 13:25:09 -0500 |
parents | 48900d625d66 |
children | d487949b2e97 |
files | src/share/vm/oops/cpCache.cpp |
diffstat | 1 files changed, 5 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/src/share/vm/oops/cpCache.cpp Tue Nov 28 12:14:30 2017 +0100 +++ b/src/share/vm/oops/cpCache.cpp Tue Dec 05 13:25:09 2017 -0500 @@ -226,14 +226,13 @@ // virtual method in java.lang.Object. This is a corner case in the spec // but is presumably legal. javac does not generate this code. // - // We set bytecode_1() to _invokeinterface, because that is the - // bytecode # used by the interpreter to see if it is resolved. + // We do not set bytecode_1() to _invokeinterface, because that is the + // bytecode # used by the interpreter to see if it is resolved. In this + // case, the method gets reresolved with caller for each interface call + // because the actual selected method may not be public. + // // We set bytecode_2() to _invokevirtual. // See also interpreterRuntime.cpp. (8/25/2000) - // Only set resolved for the invokeinterface case if method is public. - // Otherwise, the method needs to be reresolved with caller for each - // interface call. - if (method->is_public()) set_bytecode_1(invoke_code); } else { assert(invoke_code == Bytecodes::_invokevirtual, ""); }