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



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

Reply via email to