[llvm] [TableGen] Add StringInit pools to RecordKeeperImpl::dumpAllocationStats. (PR #124164)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 23 09:56:13 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-tablegen

Author: Craig Topper (topperc)

<details>
<summary>Changes</summary>

Remove duplicate print of TheBitsInitPool.

I don't know who or what uses this information. I happened to notice TheBitsInitPool was printed twice which lead to auditing the whole list.

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


1 Files Affected:

- (modified) llvm/lib/TableGen/Record.cpp (+2-1) 


``````````diff
diff --git a/llvm/lib/TableGen/Record.cpp b/llvm/lib/TableGen/Record.cpp
index b76d7bcc95a56c..590656786bc664 100644
--- a/llvm/lib/TableGen/Record.cpp
+++ b/llvm/lib/TableGen/Record.cpp
@@ -106,7 +106,8 @@ void detail::RecordKeeperImpl::dumpAllocationStats(raw_ostream &OS) const {
   OS << "TheArgumentInitPool size = " << TheArgumentInitPool.size() << '\n';
   OS << "TheBitsInitPool size = " << TheBitsInitPool.size() << '\n';
   OS << "TheIntInitPool size = " << TheIntInitPool.size() << '\n';
-  OS << "TheBitsInitPool size = " << TheBitsInitPool.size() << '\n';
+  OS << "StringInitStringPool size = " << StringInitStringPool.size() << '\n';
+  OS << "StringInitCodePool size = " << StringInitCodePool.size() << '\n';
   OS << "TheListInitPool size = " << TheListInitPool.size() << '\n';
   OS << "TheUnOpInitPool size = " << TheUnOpInitPool.size() << '\n';
   OS << "TheBinOpInitPool size = " << TheBinOpInitPool.size() << '\n';

``````````

</details>


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


More information about the llvm-commits mailing list