[llvm] TableGen: Use StringOffsetTable for RuntimeLibcall names (PR #148839)

Slava Zakharin via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 15 12:20:28 PDT 2025


vzakhari wrote:

It looks like reverting this commit requires reverting the more recent one(s).

This patch worked for me:
```
diff --git a/llvm/lib/TableGen/StringToOffsetTable.cpp b/llvm/lib/TableGen/StringToOffsetTable.cpp
index d6d2c2590389..b69db600e7c6 100644
--- a/llvm/lib/TableGen/StringToOffsetTable.cpp
+++ b/llvm/lib/TableGen/StringToOffsetTable.cpp
@@ -83,10 +83,10 @@ void StringToOffsetTable::EmitStringTableDef(raw_ostream &OS,
 #pragma GCC diagnostic pop
 #endif

-{1}constexpr llvm::StringTable
+{1} llvm::StringTable
 {2}{0} = {0}Storage;
 )",
-                Name, ClassPrefix.empty() ? "static " : "", ClassPrefix);
+                Name, ClassPrefix.empty() ? "static constexpr " : "const ", ClassPrefix);
 }

 void StringToOffsetTable::EmitString(raw_ostream &O) const {
```

If someone can please confirm that it works in at least one other case, I will create a PR for it.

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


More information about the llvm-commits mailing list