[Lldb-commits] [PATCH] D43694: Add a sanity check for inline tests

2018-02-26 Thread Phabricator via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL326140: Add a sanity check for inline testcases. (authored by adrian, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[Lldb-commits] [PATCH] D43694: Add a sanity check for inline tests

2018-02-23 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. Looks good. One extra possible check would be to make sure that at the exit of the loop, the inferior is in an expected state (eStateExited with result 0?) https://reviews.llvm.org/D43694

[Lldb-commits] [PATCH] D43694: Add a sanity check for inline tests

2018-02-23 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl created this revision. aprantl added reviewers: vsk, labath, jingham. Herald added a subscriber: eraman. When writing an inline test, there is no way to make sure that any of the inline commands are actually executed, so this patch adds a sanity check that at least one breakpoint was