Skip to content

Conversation

@ashgti
Copy link
Contributor

@ashgti ashgti commented Mar 9, 2025

This was missed in review but is showing up in lldb-dap output.

This was missed in review but is showing up in lldb-dap output.
@ashgti ashgti marked this pull request as ready for review March 9, 2025 02:52
@ashgti ashgti requested a review from JDevlieghere as a code owner March 9, 2025 02:52
@llvmbot llvmbot added the lldb label Mar 9, 2025
@llvmbot
Copy link
Member

llvmbot commented Mar 9, 2025

@llvm/pr-subscribers-lldb

Author: John Harrison (ashgti)

Changes

This was missed in review but is showing up in lldb-dap output.


Full diff: https://github.com/llvm/llvm-project/pull/130453.diff

1 Files Affected:

  • (modified) lldb/source/Host/macosx/objcxx/MemoryMonitorMacOSX.mm (-1)
diff --git a/lldb/source/Host/macosx/objcxx/MemoryMonitorMacOSX.mm b/lldb/source/Host/macosx/objcxx/MemoryMonitorMacOSX.mm
index cb6c2457df1e3..c3a430bda7fa5 100644
--- a/lldb/source/Host/macosx/objcxx/MemoryMonitorMacOSX.mm
+++ b/lldb/source/Host/macosx/objcxx/MemoryMonitorMacOSX.mm
@@ -32,7 +32,6 @@ void Start() override {
       }
     });
     dispatch_activate(m_memory_pressure_source);
-    printf("Started\n");
   }
 
   void Stop() override {

@DavidSpickett DavidSpickett merged commit 502385c into llvm:main Mar 10, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants