labath added a comment.

It looks like this test is flaky:
http://lab.llvm.org:8014/builders/lldb-x86_64-debian/builds/1339/steps/test/logs/stdio
http://lab.llvm.org:8014/builders/lldb-x86_64-debian/builds/1294/steps/test/logs/stdio

The failure messages I've seen are all the same and seem to indicate that the 
"breakpoint locations added" and "process stopped" messages come out of order. 
This seems plausible since the two messages are reported through different 
channels, and I was able to reproduce this failure locally by adding a sleep(1) 
to Debugger::HandleBreakpointEvent.

Given that the behavior of asynchronous breakpoint location messages is not the 
puprose of this test, what would you say to just deleting the "# CHECK: 1 
location added to breakpoint 1" line from the test?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61611/new/

https://reviews.llvm.org/D61611



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

Reply via email to