<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 23, 2014 at 11:22 AM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.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"><div class="">> -----Original Message-----<br>
> From: <a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:cfe-commits-">cfe-commits-</a><br>
> <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Diego Novillo<br>
> Sent: Wednesday, April 23, 2014 8:21 AM<br>
> To: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> Subject: r206995 - Review feedback<br>
...<br>
>  In particular, sample profilers can provide execution counts for all<br>
> -instructions in the code, information on branches taken and function<br>
> +instructions in the code and information on branches taken and function<br>
<br>
</div>This reads better with the comma. (It's debatable whether there should<br>
be a second comma before "and function" (the interminable "serial comma"<br>
debate)).<br></blockquote><div><br></div><div>I'm indifferent on putting a comma before the first 'and'. There should definitely not be a second comma. This is intended to parse as:</div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px"> In particular, sample profilers can provide</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px">  * execution counts for all</span><span style="font-family:arial,sans-serif;font-size:13px"> instructions in the code and</span></div><div><span style="font-family:arial,sans-serif;font-size:13px">  * information on branches taken and function</span><span style="font-family:arial,sans-serif;font-size:13px"> invocation.</span></div>
<div><br></div><div>Adding a comma before each 'and' would lead this to parse as</div><div><br></div><div><div><span style="font-family:arial,sans-serif;font-size:13px"> In particular, sample profilers can provide</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px">  * execution counts for all</span><span style="font-family:arial,sans-serif;font-size:13px"> instructions in the code, and</span></div><div><span style="font-family:arial,sans-serif;font-size:13px">  * information on branches taken, and</span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px">  * function</span><span style="font-family:arial,sans-serif;font-size:13px"> invocation.</span></div></div><div><span style="font-family:arial,sans-serif;font-size:13px"><br>
</span></div><div><span style="font-family:arial,sans-serif;font-size:13px">... which is</span> not what was intended.</div><div> <br></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">

--paulr<br>
<div class="im"><br>
>  invocation. The compiler can use this information in its optimization<br>
>  cost models. For example, knowing that a branch is taken very<br>
>  frequently helps the compiler make better decisions when ordering<br>
>  basic blocks. Knowing that a function ``foo`` is called more<br>
> -frequently than another ``bar`` helps the inliner.<br>
> +frequently than another function ``bar`` helps the inliner.<br>
<br>
<br>
<br>
</div><div class=""><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>