I have to concur with Jim's point -- writing & maintaining the gdb testsuite for years, which was based on commands & expected output like these lit tests, was a huge drag on everyone's productivity as the debugger changed over time. This style of test looks wonderfully easy to read & write & debug, but that's only if you ignore long-term maintainability, then you quickly find that it was a huge mistake. I have nothing against lit, this is a very nice way of writing commands & expected output tests, but having lived through the consequences of that style testsuite, I couldn't imagine making that choice again, especially when we've avoided it so far. It's a bit more work to read & write & debug the SB API testsuites that we have today, but it's vastly more maintainable long-term. _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
Re: [Lldb-commits] [PATCH] D24591: [LIT] First pass of LLDB LIT support
Jason Molenda via lldb-commits Wed, 14 Sep 2016 16:29:01 -0700
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Todd Fiala via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Todd Fiala via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Todd Fiala via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Sean Callanan via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Todd Fiala via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Chris Bieneman via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Enrico Granata via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Jim Ingham via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [L... Jason Molenda via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [LIT] F... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591: [L... Sean Callanan via lldb-commits
- Re: [Lldb-commits] [PATCH] D24591... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [PATCH] D2... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [PATCH] D2... Jim Ingham via lldb-commits
- Re: [Lldb-commits] [PATCH... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [... Jim Ingham via lldb-commits
- Re: [Lldb-commit... Zachary Turner via lldb-commits
- Re: [Lldb-commits] [... Jim Ingham via lldb-commits