<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 7, 2015 at 8:25 AM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">djasper added a comment.<br>
<br>
Just a minor comment about the wording of the options. Otherwise, I don't think I'll add much to the reviews by both Chandler and David.<br></blockquote><div><br></div><div>Thanks for reviewing those patches, Daniel! (Sorry for the delayed reply as I just came back from vacation).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<br>
================<br>
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:93<br>
@@ +92,3 @@<br>
+    cl::desc("Cost that models the probabalistic risk of an instruction "<br>
+             "misfetch due to a jump relative to falling through."),<br>
<span class="">+    cl::init(1), cl::Hidden);<br>
</span>----------------<br>
In what way is the cost "relative"?<br>
<br>
In my understanding, this can either mean that it is a difference or a ratio. In either case "unsigned" would be a weird data type.<br></blockquote><div><br></div><div>Here we assume that fall-through will have zero cost comparing to 1 for non-fall-through. As this is not a "relative" cost, I have updated this description to:</div><div><br></div><div><div>Cost that models the probabilistic risk of an instruction misfetch due to a jump comparing to falling through, whose cost is zero.</div></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
================<br>
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:99<br>
@@ +98,3 @@<br>
+    cl::desc(<br>
+        "Relative cost of jump instructions comparing to cost of misfethch."),<br>
<span class="">+    cl::init(1), cl::Hidden);<br>
</span>----------------<br>
Same as above, what do you actually mean by "relative"?<br></blockquote><div><br></div><div>This is a relative cost: if the cost of a misfetch is 1, then we assume the cost of a jump instruction is also 1 (we can adjust those two options to change their ratio if necessary).</div><div><br></div><div><br></div><div>Cong</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<br>
<br>
<br>
<a href="http://reviews.llvm.org/D10717" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10717</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>