<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Changes look good.<br>
<br>
Reviewed.<br>
<br>
Jon<br>
<br>
<div class="moz-cite-prefix">On 4/1/2015 4:40 AM, Stefan Karlsson
wrote:<br>
</div>
<blockquote cite="mid:551BD922.7050106@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
Hi,<br>
<br>
Please review this patch to get rid of some unnecessary calls to
oopDesc::klass(), in places where we already have the Klass
pointer.<br>
<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Estefank/8076456/webrev.01/">http://cr.openjdk.java.net/~stefank/8076456/webrev.01/</a><br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8076456">https://bugs.openjdk.java.net/browse/JDK-8076456</a><br>
<br>
This is a follow-up cleanup patch to the <a
moz-do-not-send="true"
href="https://bugs.openjdk.java.net/browse/JDK-8075955"
title="Replace the macro based implementation of oop_oop_iterate
with a template based solution">JDK-8075955</a> patch, which is
still out for review:<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017718.html">http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-March/017718.html</a><br>
<br>
Thanks,<br>
StefanK<br>
</blockquote>
<br>
</body>
</html>