llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Jacob Lalonde (Jlalond) <details> <summary>Changes</summary> In #<!-- -->98403 some of the tests were transitioned to the new `SBProcess::SaveCore(SBSaveCoreOptions)` API, but were not detected in testing. This patch addresses that. --- Full diff: https://github.com/llvm/llvm-project/pull/99692.diff 1 Files Affected: - (modified) lldb/test/API/functionalities/process_save_core/TestProcessSaveCore.py (+2-2) ``````````diff diff --git a/lldb/test/API/functionalities/process_save_core/TestProcessSaveCore.py b/lldb/test/API/functionalities/process_save_core/TestProcessSaveCore.py index 07d06bdc116ec..8573d15733927 100644 --- a/lldb/test/API/functionalities/process_save_core/TestProcessSaveCore.py +++ b/lldb/test/API/functionalities/process_save_core/TestProcessSaveCore.py @@ -20,8 +20,8 @@ def test_cannot_save_core_unless_process_stopped(self): target = self.dbg.CreateTarget(exe) process = target.LaunchSimple(None, None, self.get_process_working_directory()) self.assertNotEqual(process.GetState(), lldb.eStateStopped) - options = SBSaveCoreOptions() - options.SetOutputFile(SBFileSpec(core)) + options = lldb.SBSaveCoreOptions() + options.SetOutputFile(lldb.SBFileSpec(core)) error = process.SaveCore(core) self.assertTrue(error.Fail()) `````````` </details> https://github.com/llvm/llvm-project/pull/99692 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
