<div dir="ltr">Yea, if you see above, I mentioned that clang-format has a style option called CommentPragmas, which allows you to specify a regex for comments that clang-format won't touch.  If you specified CommentPragmas: .* then it would never touch any comment no matter what.  <div><br></div><div>(Note that I haven't actually tested this or ever used this option in practice, just see that it's there and claims to be for this use case)</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 11, 2016 at 10:00 AM Sean Callanan via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">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">In fact, the lldbinline tests could be completely broken by clang-formatting them.  They treat each //% line as a separate command to execute.  If clang-format broke those lines, lldbinline tests would stop working.<br>
<br>
Sean<br>
<br>
> On Aug 11, 2016, at 9:57 AM, Chris Lattner via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br>
><br>
><br>
>> On Aug 11, 2016, at 7:41 AM, Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br>
>><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>
> Nice!<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>
><br>
> Personally, I think that reformatting is most valuable for the debugger code itself.  The testsuite following standards seems like a separable issue, and much lower priority.<br>
><br>
> -Chris<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>
_______________________________________________<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>
</blockquote></div>