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.

Sean

> On Aug 11, 2016, at 9:57 AM, Chris Lattner via lldb-dev 
> <lldb-dev@lists.llvm.org> wrote:
> 
> 
>> On Aug 11, 2016, at 7:41 AM, Pavel Labath <lab...@google.com> wrote:
>> 
>> I just committed another  header cleanup commit, which makes lldb
>> clang-format-immune ( = it still compiles after a full reformat) on
>> linux. Other OS's are still likely to have some missed dependencies.
> 
> Nice!
> 
>> However, when I tried running the test suite I got about 150 failures.
>> Based on a sample of the errors, it looks like the problem is that
>> clang format messes up the "// Place breakpoint here" annotations we
>> use in the tests.
>> Therefore, I propose to apply the clang-format to the lldb source code
>> only as a first step. After that, as a second step, we can go through
>> the tests and fix them up so that the comment markers are where we
>> expect them to be.
> 
> 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.
> 
> -Chris
> _______________________________________________
> lldb-dev mailing list
> lldb-dev@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev

_______________________________________________
lldb-dev mailing list
lldb-dev@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev

Reply via email to