[Lldb-commits] [PATCH] D46588: [LLDB][lldb-mi] Add possibility to set breakpoints without selecting a target.
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue May 15 01:24:10 PDT 2018
labath added inline comments.
Comment at: lit/Breakpoint/Inputs/break-insert.input:1-3
Based on my experiments, lldb-mi seems to ignore lines starting with `#`. If that's true then we could put the contents of this file directly in the test, which will help readability as there is only one file to open to understand what a test does.
Comment at: lit/Breakpoint/break-insert.test:1
+# RUN: %cc %p/Inputs/break-insert.c -g
+# RUN: %lldb_mi < %p/Inputs/break-insert.input | FileCheck %s
I think the lldb-mi tests should be kept separate. Could we move this test into a different directory? Maybe `lit/tools/lldb-mi` ?
More information about the lldb-commits