Skip to content

Commit

Permalink
[lldbsuite] Fix TestBreakpointHitCount on Windows
Browse files Browse the repository at this point in the history
Summary: On Windows, the newer DIA SDKs end up producing function names that contain the return type as well. This means that the function name returned in the test will contain the return type (int) in addition to the name of the function and the type of the input (a(int)). To account for the possibility of both, the test should pass if the function name matches either pattern.

Reviewers: zturner, asmith

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D48654

llvm-svn: 335906
  • Loading branch information
sstamenova committed Jun 28, 2018
1 parent 2476374 commit aa6c3f5
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ def test_breakpoint_one_shot(self):
"There should be a thread stopped due to breakpoint")

frame0 = thread.GetFrameAtIndex(0)
self.assertEqual(frame0.GetFunctionName(), "a(int)");
self.assertTrue(frame0.GetFunctionName() == "a(int)" or frame0.GetFunctionName() == "int a(int)");

process.Continue()
self.assertEqual(process.GetState(), lldb.eStateExited)
Expand Down

0 comments on commit aa6c3f5

Please sign in to comment.