<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 26, 2013 at 10:01 PM, Mikael Lyngvig <span dir="ltr"><<a href="mailto:mikael@lyngvig.org" target="_blank">mikael@lyngvig.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">You're absolutely right (from the LR, I just searched):<div><br></div><div>  For example, if a function entry block does not have an explicit label, it will be assigned label “%0”, then the first unnamed temporary in that block will be “%1”, etc. </div>

<div><br></div><div>This is under Functions.  I guess anybody should get that far, but I personally feel that the clarification I've submitted doesn't harm.</div><span class="HOEnZb"><font color="#888888"><div><br>
</div></font></span></div></blockquote><div>Your current patch is inaccurate. If you want to duplicate what is mentioned elsewhere about the numbering to make it more likely that someone will see it, that would be OK.</div>
<div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="HOEnZb"><font color="#888888"><div></div>
<div><br></div><div>-- Mikael</div></font></span></div><div class="HOEnZb"><div class="h5">
<div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/27 Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">It is correct as written. Unnamed BB's are "unnamed temporaries" from a counting perspective. If the first BB is unlabeled, then it becomes %0. The first instruction then becomes %1, etc.<div>

<br>
</div><div>I swear we mention that somewhere in LangRef (I seem to recall reviewing a patch mentioning that), but if not, a patch mentioning that would be fine.<div><br></div><div>-- Sean Silva</div></div></div><div class="gmail_extra">


<br><br><div class="gmail_quote">On Tue, Nov 26, 2013 at 8:50 PM, Mikael Lyngvig <span dir="ltr"><<a href="mailto:mikael@lyngvig.org" target="_blank">mikael@lyngvig.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div dir="ltr">This patch adds the following sentence to the LR after it states that temporaries begin at index 0:<div><br><div>   If you do not specify a label for the entry block, the temporaries start at index 1 as index<br>



</div><div>   0 is then used for an implicit label generated by LLVM.</div><div><br></div><div>This because LLC complains when you use %0 as a temporary and there is no entry point label in the function.  Then you get an error message, which can be quite difficult to figure out why.</div>



</div><div><br></div><div>If this patch is rejected, somebody should probably phrase the above appropriately and submit a new patch.</div><span><font color="#888888"><div><br></div><div><br></div><div>-- Mikael</div>
<div><br></div></font></span></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>