jingham added a comment.

In D85106#2194840 <https://reviews.llvm.org/D85106#2194840>, @clayborg wrote:

> lldb/test/API is usually for testing the lldb::SB* interfaces IIRC. So not 
> sure this move makes sense?

It's really for any test that wants to use the lldbtest test harness.  There 
are other tests in the commands sub-directory that use lldbutil methods to get 
the test up and going, then a bunch of self.expects to test various command 
line output.  Oddly enough there are also some tests in there (like the 
disassemble/basic test) that don't seem to test command line commands at all.  
But that's a separate issue.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85106/new/

https://reviews.llvm.org/D85106

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

Reply via email to