<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 1, 2014 at 9:07 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.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"><br>
On 02/07/2014 05:48, Sean Silva wrote:<div><div class="h5"><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">
<br>
<br>
<br>
On Tue, Jul 1, 2014 at 6:41 PM, Matt Arsenault <<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.com</a> <mailto:<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.<u></u>com</a>>> wrote:<br>

<br>
    On 07/01/2014 05:31 PM, Reid Kleckner wrote:<br>
<br>
        Sounds good to me, go for it.  I'd wordsmith it a bit.  I<br>
        don't think "is basically free" is important.  The important<br>
        thing is that the stack adjustment is folded into the function<br>
        prologue.<br>
<br>
    Both of these are very target specific details that don't really<br>
    have anything to do with the IR instruction's semantics<br>
<br>
<br>
I agree that it is target-specific, and that target-specific information should be avoided in LangRef. However, this is a fairly important distinction; for example, a frontend author should definitely be aware of it.<br>

<br>
I'll wait for some more opinions.<br>
</blockquote>
<br></div></div>
Seems fine to add this, but perhaps drop the loud "Note" heading</blockquote><div><br></div><div>There is already precedent to create sectional divisions for remarks that don't fall into the "standard" headings of the instructions. E.g. <a href="http://llvm.org/docs/LangRef.html#implementation">http://llvm.org/docs/LangRef.html#implementation</a></div>
<div>(gah that anchor name is really bad...)</div><div>Also there is precedent for removing them as needed e.g. <a href="http://llvm.org/docs/LangRef.html#unreachable-instruction">http://llvm.org/docs/LangRef.html#unreachable-instruction</a></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"> and qualify the statement with something like "a backend may fold" to address Matt's observation.<br>
</blockquote><div><br></div><div>That sounds like a good thing to call out explicitly.</div><div><br></div><div>-- Sean Silva</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>
Alp.<div class=""><div class="h5"><br>
<br>
<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">
<br>
-- Sean Silva<br>
<br>
<br>
______________________________<u></u>_________________<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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>
<br></div></div><span class=""><font color="#888888">
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</font></span></blockquote></div><br></div></div>