[Lldb-commits] [PATCH] D33426: Introduce new command: thread backtrace unique
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed May 31 03:44:04 PDT 2017
labath added a comment.
In https://reviews.llvm.org/D33426#766525, @bgianfo wrote:
> Address Pavel and Greg's feedback on Diff 100365.
> Pavel: I took your suggestions to make the test case more readable,
> I really appreciate the guidance. I did have to tweak some of the
> functionality to make the test case pass reliably, as there were
> still some races possible. I also saw that SBThread.Resume() seems
> to occasionally result in a StopReason of eStopReasonNon. So I worked
> around that by only including threads int expected output that the Resume
> resulted in making it to our breakpoint. I have verified the test is
> consistently passes by executing it on repeat 100 times,
Thanks. The fact that we are not able to rely on the operation of Resume in this case sounds like a bug. Obviously we can't condition the acceptance of this patch by fixing that issue, but we should at least track it. Can you create a bug on bugs.llvm.org, and reference it in your workaround. BTW, what's the platform you are testing this on?
More information about the lldb-commits