[Lldb-commits] [PATCH] D54680: Don't use lldb -O in lit tests

Frederic Riss via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sun Nov 18 18:23:24 PST 2018


friss added a comment.

I do agree it's slightly easier to read this way. I was conflicted while writing the tests between readability and conciseness. I think this is a good compromise.

I do prefer to have the check lines with the commands rather than with the source code though. The output is the result of the commands and sometimes it's not as clear what it refers to in the source.


https://reviews.llvm.org/D54680





More information about the lldb-commits mailing list