<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 26, 2014 at 6:47 PM, Philip Reames <span dir="ltr"><<a href="mailto:listmail@philipreames.com" target="_blank">listmail@philipreames.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch is a follow up to the thread "Stale type system documentation?" on llvmdev from a few weeks ago.<br>

<br>
Two important notes:<br>
- I don't have commit access and will need someone else to commit this patch<br>
- I couldn't figure out how to run the actual HTML generation step. I've tried to be careful, but you should check the change actually builds properly.<br></blockquote><div><br></div><div>See docs/README.txt for how to generate the HTML (if that's not enough, let me know and I will improve it to address whatever difficulties you ran into). Off the top of my head, I at least see one issue which is that in constructs like :ref:`Foo <Bar>`, you need to use backticks instead of single quotes.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Patch Comment:<br>
This patch removes confusing and out of date text from the LangRef. LLVM no longer performs structural uniquing on named types.  Most of the documentation was updated to reflect this, but this section got missed.  I attempted to replace the original wording with appropriate forward references, but it's debatable whether the section should be kept at all.</blockquote>
<div><br></div><div>I think you have the direction of this patch backward. AFAIK the *current* system uses structural uniquing.</div><div><br></div><div>-- Sean Silva</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
<br>
Philip<br>
</font></span></blockquote></div><br></div></div>