<div dir="ltr">I was thinking the same thing too.  I figured this was just for the interim.<div><br></div><div>Chris, did you mean to update the global LLVM style conventions?</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 11, 2016 at 11:27 AM Jim Ingham <<a href="mailto:jingham@apple.com">jingham@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Shouldn't this be made general and added to the llvm coding conventions?  I was assuming that upon completion of this exercise, we would delete the lldb coding conventions doc.<br>
<br>
Jim<br>
<br>
> On Aug 11, 2016, at 11:20 AM, Zachary Turner via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br>
><br>
> On Wed, Aug 10, 2016 at 10:37 PM Chris Lattner <<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>> wrote:<br>
><br>
>> 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:<br>
>><br>
>> So perhaps it would be reasonable for us to standardize on something like this:<br>
>><br>
>>      • Main Module Header<br>
>>      • Local/Private Headers<br>
>>      • lldb/...<br>
>>      • llvm/...<br>
>>      • System #includes<br>
><br>
> 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.<br>
><br>
> -Chris<br>
><br>
> 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>
><br>
> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
<br>
</blockquote></div>