polyakov.alex added inline comments.

================
Comment at: lit/tools/lldb-mi/breakpoint/break-insert.test:10
+-file-exec-and-symbols a.out
+# CHECK-AFTER: ^done
+
----------------
labath wrote:
> polyakov.alex wrote:
> > 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.
You are right about the CHECK-AFTER. I wrote the test using only check and 
found an issue:
after executing of -exec-run command, we expect to see output like: 
*stopped,reason="breakpoint-hit"
but, I think, that FileCheck check file for the pattern, while a -exec-run 
hasn't finished yet. It means that there will not be expected output and we'll 
get the error: no such pattern.

Is there a mechanism to add some delay to FileCheck to wait until -exec-run 
finished?



Repository:
  rL LLVM

https://reviews.llvm.org/D46588



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

Reply via email to