[Lldb-commits] [lldb] [lldb] Add count for number of DWO files loaded in statistics (PR #144424)

[David Peixotto via lldb-commits]

================
@@ -512,6 +517,32 @@ def test_breakpoints(self):
             self.verify_keys(
                 breakpoint, 'target_stats["breakpoints"]', bp_keys_exist, None
             )
+    
+    def test_loaded_dwo_file_count(self):
+        """
+        Test "statistics dump" and the loaded dwo file count.
+        Builds a binary w/ separate .dwo files and debug_names, and then
+        verifies the loaded dwo file count is the expected count after running
+        various commands
+        """
+        da = {"CXX_SOURCES": "third.cpp baz.cpp", "EXE": self.getBuildArtifact("a.out")}
+        self.build(dictionary=da, debug_info=["dwo", "debug_names"])
----------------
dmpots wrote:

I see. So debug names is used to prevent the eager loading of DWO files. Is debug names the best way to do that? Or is there a more direct setting (like `target.preload-symbols`) that we could use instead. I don't know the answer.

At least it would be good to add a comment about the reason we are using debug names in this test.

https://github.com/llvm/llvm-project/pull/144424