[llvm] [RISC-V] Pool duplicate scheduling class descriptors (PR #202659)

[Alexis Engelke via llvm-commits]

================
@@ -1391,6 +1457,9 @@ void SubtargetEmitter::emitSchedClassTables(SchedClassTables &SchedTables,
   StringToOffsetTable StrTab;
   unsigned InvalidNameOff = StrTab.GetOrAddStringOffset("InvalidSchedClass");
 
+  if (SchedTables.UseSchedClassPool)
+    OS << "\n#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)\n";
----------------
aengelke wrote:

I don't think we should go down this route, different tablegen headers, that themselves aren't quite easy to follow, and data structures in release builds just wait for bugs to happen and will be really hard to debug...

Can we perhaps store the name offset at the point where the model is referenced?

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