[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 04:00:50 PDT 2018
labath added inline comments.
Comment at: lit/tools/lldb-mi/breakpoint/break-insert.test:10
+# CHECK-AFTER: ^done
> labath wrote:
> > I'm not familiar with this directive. Are you sure that this actually does anything?
> I tried to use only CHECK directive, but got errors like pattern not found, so I decided that it may be caused due to CHECK search features, for example, as I know, it finds pattern from the start of the file. If we want to check lldb-mi output, we should follow a specific order.
> In our case, we should find "^done" string directly after -break-insert command's output.
I think you got the FileCheck operation wrong.
a `CHECK` should always start matching from the previous match. The reason that this is passing for you now is that CHECK-AFTER is a non-existing directive and FileCheck ignores it (try replacing it with a bogus string and see if it still passes). if `CHECK` is not working for you here then you probably have the pattern wrong.
More information about the lldb-commits