<div dir="ltr">I assume Christian Convey was referring to clang-format moving the "//breakpoint here" comments in the tests to different lines.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 11, 2016 at 8:15 AM, Zachary Turner via lldb-dev <span dir="ltr"><<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It's not possible.  The problem is that lldb was dependent on order of includes because each header wasn't properly including what it used.  So when clang-format reordered this, things broke <br><div class="HOEnZb"><div class="h5"><div class="gmail_quote"><div dir="ltr">On Thu, Aug 11, 2016 at 8:04 AM Christian Convey via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Pavel,<br>
<br>
Would it make sense to address this problem by fixing clang-format,<br>
rather than working around it?<br>
<br>
(Assuming the clang-format fix is relatively easy, and acceptable to<br>
clang-format's maintainers.)<br>
<br>
- Christian<br>
<br>
On Thu, Aug 11, 2016 at 10:41 AM, Pavel Labath via lldb-dev<br>
<<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br>
> I just committed another  header cleanup commit, which makes lldb<br>
> clang-format-immune ( = it still compiles after a full reformat) on<br>
> linux. Other OS's are still likely to have some missed dependencies.<br>
><br>
> However, when I tried running the test suite I got about 150 failures.<br>
> Based on a sample of the errors, it looks like the problem is that<br>
> clang format messes up the "// Place breakpoint here" annotations we<br>
> use in the tests.<br>
> Therefore, I propose to apply the clang-format to the lldb source code<br>
> only as a first step. After that, as a second step, we can go through<br>
> the tests and fix them up so that the comment markers are where we<br>
> expect them to be.<br>
______________________________<wbr>_________________<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/<wbr>mailman/listinfo/lldb-dev</a><br>
</blockquote></div>
</div></div><br>______________________________<wbr>_________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org">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/<wbr>mailman/listinfo/lldb-dev</a><br>
<br></blockquote></div><br></div>