<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Jan 15, 2014, at 6:49 PM, Christian Thalinger &lt;<a href="mailto:christian.thalinger@oracle.com">christian.thalinger@oracle.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span style="font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; display: inline !important; float: none; ">John would know the answer.</span><br style="font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><br style="font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><span style="font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; display: inline !important; float: none; ">Given this change should go into JDK 8 I think we should push for now. &nbsp;If we can come up with a better way to handle these situations we can push another change for 9 or 8u20.</span><br style="font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "></blockquote></div><br><div>(Back from vacation.)</div><div><br></div><div>Vladimir's change for bytecode generation is safe. &nbsp;It assumes something stable, which is that virtual invocation mode can apply to interfaces only because they inherit Object. &nbsp;Therefore we can back off from invokevirtual to invokeinterface, without changing behavior.</div><div><br></div><div>FTR, this fix may be redundant with this changeset:</div><div>&nbsp; <a href="http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/9063bd8808a7">http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/9063bd8808a7</a></div><div><br></div><div>...and specifically with the line of code which says "m_klass = m_klass_non_interface". &nbsp;This redundancy is of the belt-and-suspenders kind; it could be changed to an assert, but is fine the way it is.</div><div><br></div><div>Thanks for fixing this!</div><div><br></div><div> John</div></body></html>