<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Thank you Christian,<br>
<br>
//nils<br>
<br>
<div class="moz-cite-prefix">On 2015-11-12 19:01, Christian
Thalinger wrote:<br>
</div>
<blockquote
cite="mid:4DB47AB9-82B2-4689-9801-93C100EFD10F@oracle.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
Looks good.
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On Nov 12, 2015, at 3:10 AM, Nils Eliasson
<<a moz-do-not-send="true"
href="mailto:nils.eliasson@oracle.com" class="">nils.eliasson@oracle.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252" class="">
<div text="#000000" bgcolor="#FFFFFF" class=""> Hi,<br
class="">
<br class="">
Please review this small fix.<br class="">
<br class="">
Two issues where identified when investigating this
crash:<br class="">
1) In CompileBroker a methodHandle was not used when
looking up the directive<br class="">
2) In DirectivesStack::getMatchingDirective the
directivesStack::_top was accessed before taking the
DirectivesStack_lock<br class="">
<br class="">
bug: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8141585">https://bugs.openjdk.java.net/browse/JDK-8141585</a><br
class="">
webrev: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eneliasso/8141585/webrev.01/">http://cr.openjdk.java.net/~neliasso/8141585/webrev.01/</a><br
class="">
<br class="">
Regards,<br class="">
Nils Eliasson<br class="">
<br class="">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252" class="">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252" class="">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252" class="">
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</blockquote>
<br>
</body>
</html>