<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Aug 10, 2016 at 10:37 PM Chris Lattner <<a href="mailto:clattner@apple.com">clattner@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><blockquote type="cite"><div>On Aug 9, 2016, at 3:01 PM, Zachary Turner via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:</div><br class="m_5692857173949537652Apple-interchange-newline"><div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">So perhaps it would be reasonable for us to standardize on something like this:</div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><br></div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><ol class="m_5692857173949537652inbox-inbox-inbox-inbox-arabic m_5692857173949537652inbox-inbox-inbox-inbox-simple" id="m_5692857173949537652inbox-inbox-inbox-inbox-local-private-headers" style="font-family:'lucida grande','lucida sans unicode',geneva,verdana,sans-serif;font-size:14px;line-height:21px"><li>Main Module Header</li><li>Local/Private Headers<br></li><li><code class="m_5692857173949537652inbox-inbox-inbox-inbox-inbox-inbox-literal m_5692857173949537652inbox-inbox-inbox-inbox-inbox-inbox-docutils" style="font-family:consolas,'deja vu sans mono','bitstream vera sans mono',monospace;font-size:0.95em"><span class="m_5692857173949537652inbox-inbox-inbox-inbox-inbox-inbox-pre">lldb/...</span></code></li><li><code class="m_5692857173949537652inbox-inbox-inbox-inbox-docutils m_5692857173949537652inbox-inbox-inbox-inbox-literal" style="font-family:consolas,'deja vu sans mono','bitstream vera sans mono',monospace;font-size:0.95em"><span class="m_5692857173949537652inbox-inbox-inbox-inbox-pre">llvm/...</span></code></li><li>System<span class="m_5692857173949537652inbox-inbox-inbox-inbox-Apple-converted-space"> </span><code class="m_5692857173949537652inbox-inbox-inbox-inbox-docutils m_5692857173949537652inbox-inbox-inbox-inbox-literal" style="font-family:consolas,'deja vu sans mono','bitstream vera sans mono',monospace;font-size:0.95em"><span class="m_5692857173949537652inbox-inbox-inbox-inbox-pre">#include</span></code>s</li></ol></div></div></blockquote><br></div></div><div style="word-wrap:break-word"><div>This makes sense to me, and matches what clang does as well.  I think that this is clearly in the spirit of the llvm include order standards, and I think it would be great to make this explicit in the coding standard doc.  Can you send in a patch to update it to make this explicit?  I’ll review it.</div></div><div style="word-wrap:break-word"><div><br></div><div>-Chris</div></div></blockquote><div> </div>I actually just submitted the patch.  (Sorry, itchy trigger finger or something).  r278373.  If you have any comments let me know and I'm happy to iterate on it.<br><div> </div></div></div>