<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;" class="">Hi Vladimir,<div class=""><br class=""></div><div class="">thank you.</div><div class=""><br class=""></div><div class="">Anyone, please review ...</div><div class=""><br class=""></div><div class="">Best,</div><div class=""><br class=""></div><div class="">Michael</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">Am 30.07.2015 um 04:15 schrieb Vladimir Kozlov <<a href="mailto:vladimir.kozlov@oracle.com" class="">vladimir.kozlov@oracle.com</a>>:</div><br class="Apple-interchange-newline"><div class="">This looks good to me. You need secondreviewer to look on this since changes are big.<br class=""><br class="">Thanks.<br class="">Vladimir<br class=""><br class="">On 7/29/15 1:54 AM, Michael Haupt wrote:<br class=""><blockquote type="cite" class="">Hi Vladimir,<br class=""><br class="">thank you for your comments. I have uploaded a revised webrev to<br class=""><a href="http://cr.openjdk.java.net/~mhaupt/8004073/webrev.01" class="">http://cr.openjdk.java.net/~mhaupt/8004073/webrev.01</a>; some replies are<br class="">inlined below.<br class=""><br class=""><blockquote type="cite" class="">Am 28.07.2015 um 19:24 schrieb Vladimir Kozlov<br class=""><<a href="mailto:vladimir.kozlov@oracle.com" class="">vladimir.kozlov@oracle.com</a> <<a href="mailto:vladimir.kozlov@oracle.com" class="">mailto:vladimir.kozlov@oracle.com</a>>>:<br class="">Before looking to webrev, can you use whole word Node::related(),<br class="">dump_related(), dump_related_compact(), dump_compact()? "comp" could<br class="">be confused for "compiled". It is more typing in debugger but it is<br class="">more clear.<br class=""></blockquote><br class="">Done.<br class=""><br class=""><blockquote type="cite" class="">Also from  this->dump_rel() in your example I see that you dump a lot<br class="">more input nodes than I expect (only up to inputs of CmpU node).<br class="">But this->dump_rel_comp() produces correct set of nodes.<br class=""></blockquote><br class="">The depth of output can be controlled with the method<br class="">Node::dump_related(int d_in, int d_out); in my initial post I had not<br class="">mentioned this method. The default output is also formatted in a way<br class="">that makes clear where the current node (>) is, and where all the inputs<br class="">(before) and outputs (after) are. Regarding the notion of "related<br class="">nodes", YMMV.<br class=""><br class="">For additional illustration, I've added an implementation of related()<br class="">for PhiNode.<br class=""><br class=""><blockquote type="cite" class="">It would be nice if you can avoid using macro:<br class=""><br class="">+#ifndef PRODUCT<br class="">+  REL_IN_DATA_OUT_1;<br class="">+#endif<br class=""><br class="">"Arithmetic nodes" are most common data nodes (vs control nodes<br class="">this->is_CFG() == true). May be instead specialized rel() method you<br class="">can use some flags checks in Node::rel() method.<br class=""></blockquote><br class="">Done.<br class=""><br class="">Best,<br class=""><br class="">Michael</blockquote></div></blockquote><br class=""></div><br class=""><div apple-content-edited="true" class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">-- <br class=""><br class=""><a href="http://www.oracle.com/" target="_blank" class=""><img src="http://www.oracle.com/dm/design/images/oracle_sig_logo.gif" alt="Oracle" width="114" height="26" border="0" class=""></a><br class=""><font color="#666666" size="2" face="Verdana, Arial, Helvetica, sans-serif" class="">Dr. Michael Haupt | Principal Member of Technical Staff<br class="">Phone: +49 331 200 7277 | Fax: +49 331 200 7561<br class=""><font color="#FF0000" class="">Oracle</font><span class="Apple-converted-space"> </span>Java Platform Group | LangTools Team<span class="Apple-converted-space"> | Nashorn</span><br class="">Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany</font><table width="100%" border="0" cellpadding="0" cellspacing="0" class=""><tbody class=""><tr class=""><td width="44" height="0" align="left" valign="top" class=""><a href="http://www.oracle.com/commitment" target="_blank" class=""><img src="http://www.oracle.com/dm/design/corp/misc/green-for-email-sig_0.gif" alt="Green Oracle" width="44" height="28" border="0" align="absbottom" class=""></a></td><td width="100%" align="left" valign="middle" class=""><font color="#47AA42" size="1" face="Verdana, Arial, Helvetica, sans-serif" class=""><font color="#4B7D42" class="">Oracle is committed to developing practices and products that help protect the environment</font></font></td></tr></tbody></table></div></span></div>
</div>
<br class=""></div></body></html>