<div dir="ltr"><br><div class="gmail_extra">On Mon, May 27, 2013 at 7:36 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<div class="gmail_quote"><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="im">> Ah, ok. In that case, I think that it would be best to make a new page for<br>

> libSupport, and have it defer to SystemLibrary.rst for discussion of the<br>
> "libSystem" parts of libSupport. The major necessary changes for<br>
> SystemLibrary.rst would then be to mention its inclusion in libSupport<br>
> (important) and fix file paths (mechanical, less important).<br>
<br>
</div>Sorry, but at least for me having a docs/SystemLibrary.rst and so<br>
lib/System is very confusing.</blockquote><div><br></div><div style>As I mentioned, for the moment the page should (probably in its first sentence) mention that the code has been merged into libSupport and that it doesn't exist in the tree as lib/System. Simply replacing "System" with "Support" doesn't really buy anything, besides misrepresenting what libSupport actually is (consider the second sentence of <<a href="http://llvm.org/docs/SystemLibrary.html">http://llvm.org/docs/SystemLibrary.html</a>> (describing the purpose), which is not accurate about libSupport as a whole).</div>
<div style><br></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">Ideally we would have a<br>

docs/SystemLibrary.rst that would just says "this library has been<br>
merged to lib/Support" and docs/SupportLibrary.rst documents whatever<br>
is in lib/Support.<br></blockquote><div><br></div><div>Considering our OS portability layer to be it's own separate thing, even if it isn't its own lib/* directory is probably a good distinction to make regardless. And SystemLibrary.rst is well-written and has excellent, focused content about LLVM's approach to OS portability. </div>
<div><br></div><div>After thinking about this a bit more, it's not clear to me that it would be beneficial to include this content into a general page about libSupport, as that would make it less focused and harder to find. If anything, it would be "ideal" to put it into a file Portability.rst (or similar), but that's a marginal benefit anyway since it is already one of the top hits when searching "llvm portability". We can really easily massage the title and content (such as referenced file paths), like what happended to clang/docs/Tooling.rst, which is now "Choosing the Right Interface for Your Application" <<a href="http://clang.llvm.org/docs/Tooling.html">http://clang.llvm.org/docs/Tooling.html</a>>.</div>
<div><br></div><div style>-- Sean Silva </div></div></div></div>