[Lldb-commits] [lldb] [LLDB][Data Formatters] Calculate average and total time for summary providers within lldb (PR #102708)

via lldb-commits lldb-commits at lists.llvm.org
Fri Aug 9 17:40:23 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {darker}-->


:warning: Python code formatter, darker found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
darker --check --diff -r a3ccaed3b9f6a1fe9b7f2ef019259f88072639b2...ad4b20d2074203fce9e4c47fd6045699b1528d86 lldb/test/API/commands/statistics/basic/TestStats.py
``````````

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- TestStats.py	2024-08-10 00:36:10.000000 +0000
+++ TestStats.py	2024-08-10 00:39:57.580943 +0000
@@ -248,11 +248,11 @@
             "firstStopTime",
             "frameVariable",
             "launchOrAttachTime",
             "moduleIdentifiers",
             "targetCreateTime",
-            "summaryProviderStatistics"
+            "summaryProviderStatistics",
         ]
         self.verify_keys(stats, '"stats"', keys_exist, None)
         self.assertGreater(stats["firstStopTime"], 0.0)
         self.assertGreater(stats["launchOrAttachTime"], 0.0)
         self.assertGreater(stats["targetCreateTime"], 0.0)
@@ -446,11 +446,11 @@
             "expressionEvaluation",
             "frameVariable",
             "targetCreateTime",
             "moduleIdentifiers",
             "totalBreakpointResolveTime",
-            "summaryProviderStatistics"
+            "summaryProviderStatistics",
         ]
         self.verify_keys(target_stats, '"stats"', keys_exist, None)
         self.assertGreater(target_stats["totalBreakpointResolveTime"], 0.0)
         breakpoints = target_stats["breakpoints"]
         bp_keys_exist = [
@@ -918,14 +918,14 @@
             self.assertEqual(
                 debug_stats_0,
                 debug_stats_1,
                 f"The order of options '{options[0]}' and '{options[1]}' should not matter",
             )
-    
+
     def test_summary_statistics_providers(self):
         """
-        Test summary timing statistics is included in statistics dump when 
+        Test summary timing statistics is included in statistics dump when
         a type with a summary provider exists, and is evaluated.
         """
 
         self.build()
         target = self.createTestTarget()

``````````

</details>


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


More information about the lldb-commits mailing list