[Lldb-commits] [PATCH] D43419: Fix TestBreakpointInGlobalConstructor for Windows
Adrian McCarthy via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 16 15:53:27 PST 2018
amccarth created this revision.
amccarth added a reviewer: jasonmolenda.
Herald added a subscriber: sanjoy.
This test was failing on Windows because it expected the breakpoint in the
dynamic library to be resolved before the process is launched. Since the DLL
isn't loaded until the process is launched this didn't work.
Changing the expectation solves the problem on Windows, but since I'm not
sure how this works on other platforms, I'm afraid this could break it there.
I'm considering whether we should treat 0 as a special value for
`num_expected_locations` that means 0 or more.
@@ -29,8 +29,11 @@
bp_main = lldbutil.run_break_set_by_file_and_line(
self, 'main.cpp', self.line_main)
+ # We don't expect any locations because the dynamic library isn't
+ # loaded until we launch the process.
bp_foo = lldbutil.run_break_set_by_file_and_line(
- self, 'foo.cpp', self.line_foo)
+ self, 'foo.cpp', self.line_foo, num_expected_locations=0)
process = target.LaunchSimple(
None, env, self.get_process_working_directory())
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 971 bytes
Desc: not available
More information about the lldb-commits