Ping.<div><br></div><div>No changes to Datalayout.cpp between the 4th and now, so the patch should still be good. (As long as my interpretation of DataLayout::init() is right!)<div><div><br><div class="gmail_quote">On 4 January 2013 19:23, Stephen McGruer <span dir="ltr"><<a href="mailto:stephen.mcgruer@gmail.com" target="_blank">stephen.mcgruer@gmail.com</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">(Ping, and changed subject to llvm-commits style.)<div><br></div><div>This patch updates the LangRef documentation to correctly describe the default datalayout string. The current documentation (<a href="http://llvm.org/docs/LangRef.html#data-layout" target="_blank">http://llvm.org/docs/LangRef.html#data-layout</a>) and DataLayout::init() (<a href="http://llvm.org/docs/doxygen/html/DataLayout_8cpp_source.html#l00154" target="_blank">http://llvm.org/docs/doxygen/html/DataLayout_8cpp_source.html#l00154</a>) disagree on a number of points.</div>

<div><br></div><div>Can someone check that my interpretation of DataLayout::init() is correct and (assuming so) commit the patch?</div><div><br></div><div>A full description can be found at <a href="http://llvm.org/bugs/show_bug.cgi?id=14673" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14673</a>.</div>

<div><br></div><div>Stephen</div><div><br></div><div><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Stephen McGruer</b> <span dir="ltr"><<a href="mailto:stephen.mcgruer@gmail.com" target="_blank">stephen.mcgruer@gmail.com</a>></span><br>

Date: 21 December 2012 14:26<br>Subject: Patch for bug 14673 (Mismatchs between Datalayout documentation and code)<br>To: <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br><br><br>
<div dir="ltr">Dear all,<div>
<br></div><div>This patch updates the LangRef documentation to correctly describe the default datalayout string in LLVM. The current listing has a number of inaccuracies, the most important of which is probably the mistaken statement that aggregate alignment defaults to a0:0:1, instead of the true a0:0:64.</div>


<div><br></div><div>Full description at <a href="http://llvm.org/bugs/show_bug.cgi?id=14673" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14673</a>.</div><span class="HOEnZb"><font color="#888888"><span><font color="#888888"><div>
<br></div><div>Stephen</div>
</font></span></font></span></div>
</div><br></div></div>
</blockquote></div><br></div></div></div>