Address David's comments

Author: royitaqi

lldb/source/Target/Statistics.cpp

@@ -323,7 +323,7 @@ llvm::json::Value DebuggerStats::ReportStatistics(
     if (symtab) {
       module_stat.num_symbols_loaded = symtab->GetNumSymbols();
       num_symbols_loaded += module_stat.num_symbols_loaded;
-      symtabs_loaded++;
+      ++symtabs_loaded;
       module_stat.symtab_loaded_from_cache = symtab->GetWasLoadedFromCache();
       if (module_stat.symtab_loaded_from_cache)
         ++symtabs_loaded_from_cache;

lldb/test/API/commands/statistics/basic/TestStats.py

@@ -171,13 +171,16 @@ def test_default_no_run(self):
             "totalSymbolTablesLoadedFromCache",
             "totalSymbolTablesSavedToCache",
             "totalSymbolsLoaded",
+            "totalSymbolTablesLoaded",
             "totalDebugInfoByteSize",
             "totalDebugInfoIndexTime",
             "totalDebugInfoIndexLoadedFromCache",
             "totalDebugInfoIndexSavedToCache",
             "totalDebugInfoParseTime",
         ]
         self.verify_keys(debug_stats, '"debug_stats"', debug_stat_keys, None)
+        self.assertGreater(debug_stats["totalSymbolsLoaded"], 0)
+        self.assertGreater(debug_stats["totalSymbolTablesLoaded"], 0)
 
         # Verify target stats keys.
         target_stats = debug_stats["targets"][0]
@@ -188,16 +191,23 @@ def test_default_no_run(self):
             "targetCreateTime",
         ]
         target_stat_keys_missing = ["firstStopTime", "launchOrAttachTime"]
-        self.verify_keys(target_stats, '"target_stats"', target_stat_keys_exist, target_stat_keys_missing)
+        self.verify_keys(
+            target_stats,
+            '"target_stats"',
+            target_stat_keys_exist,
+            target_stat_keys_missing,
+        )
         self.assertGreater(target_stats["targetCreateTime"], 0.0)
 
         # Verify module stats keys.
         for module_stats in debug_stats["modules"]:
             module_stat_keys_exist = [
                 "symbolsLoaded",
             ]
-            self.verify_keys(module_stats, '"module_stats"', module_stat_keys_exist, None)
-
+            self.verify_keys(
+                module_stats, '"module_stats"', module_stat_keys_exist, None
+            )
+            self.assertGreater(module_stats["symbolsLoaded"], 0)
 
     def test_default_with_run(self):
         """Test "statistics dump" when running the target to a breakpoint.