[TableGen] Increase IndexWidth DAGISelMatcherEmitter.

[topperc]

RISC-V has over a million bytes in the table.

[llvmbot]

@llvm/pr-subscribers-llvm-selectiondag

@llvm/pr-subscribers-tablegen

Author: Craig Topper (topperc)

Changes

RISC-V has over a million bytes in the table.

---

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

1 Files Affected:

• (modified) llvm/utils/TableGen/DAGISelMatcherEmitter.cpp (+1-1)

diff --git a/llvm/utils/TableGen/DAGISelMatcherEmitter.cpp b/llvm/utils/TableGen/DAGISelMatcherEmitter.cpp
index d3fba6a92357a..ee10500a7ff50 100644
--- a/llvm/utils/TableGen/DAGISelMatcherEmitter.cpp
+++ b/llvm/utils/TableGen/DAGISelMatcherEmitter.cpp
@@ -29,7 +29,7 @@
 using namespace llvm;
 
 enum {
-  IndexWidth = 6,
+  IndexWidth = 7,
   FullIndexWidth = IndexWidth + 4,
   HistOpcWidth = 40,
 };

[s-barannikov]

LGTM

As a suggestion, I think the value could be computed from the table size. So that we don't print redundant spaces.

[topperc]

LGTM

As a suggestion, I think the value could be computed from the table size. So that we don't print redundant spaces.

Yeah. The calculation in your decoder patch reminded I had been meaning to update the number. The comments in DAG aren't enabled by default so you only see any of this if you add the cmake flag or run tblgen manually.