[Lldb-commits] [PATCH] D16858: Add verbose logging support to gdb-remote tests

[Todd Fiala via lldb-commits]

tfiala added a comment.

In http://reviews.llvm.org/D16858#343895, @labath wrote:

> In http://reviews.llvm.org/D16858#343140, @tfiala wrote:
>
> > BTW I was planning on re-organizing the lldb-server/debugserver tests sometime in the near future to break them into many different directories and pull off the dsym/dwarf/dwo changes for the many tests where that doesn't matter.
>
>
> I think that basically none of these tests rely on debug info, and we can remove the debug info duplication entirely. Unfortunately, the only way right now is to annotate each function with `@no_debug_info_test`, so I have been waiting until I get a chance to make that decorator work on classes.
>
> That should cut the running time in half, so then you may not even need to split stuff out more...


True.

Maybe I'll wait until then.  If/when I get to writing more tests in that area, I may reorganize just because it is currently a massive hunk of tests in one file, but if you're going to get rid of the alternatives with a class-level decorator, that definitely will remove the main piece I cared about (namely, the time).


Repository:
  rL LLVM

http://reviews.llvm.org/D16858