Revert r152105: "Use uint16_t to store indices into string table"
authorJakob Stoklund Olesen <stoklund@2pi.dk>
Thu, 15 Mar 2012 18:05:54 +0000 (18:05 +0000)
committerJakob Stoklund Olesen <stoklund@2pi.dk>
Thu, 15 Mar 2012 18:05:54 +0000 (18:05 +0000)
This patch limited the concatenated register names to 64K which meant
that the total number of registers was many times less than 64K.

If any compilers actually enforce the 64K limit on string literals, and
it turns out to be a problem, we should fix that problem by not using
long string literals.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@152816 91177308-0d34-0410-b5e6-96231b3b80d8

utils/TableGen/AsmWriterEmitter.cpp

index 7671efdde0a3632a38a96643aa4fbf78a95183e1..0ef5a93d569fd469f4462573dfceddce57d526fb 100644 (file)
@@ -464,7 +464,7 @@ static void
 emitRegisterNameString(raw_ostream &O, StringRef AltName,
   const std::vector<CodeGenRegister*> &Registers) {
   StringToOffsetTable StringTable;
-  O << "  static const uint16_t RegAsmOffset" << AltName << "[] = {\n    ";
+  O << "  static const unsigned RegAsmOffset" << AltName << "[] = {\n    ";
   for (unsigned i = 0, e = Registers.size(); i != e; ++i) {
     const CodeGenRegister &Reg = *Registers[i];
 
@@ -510,7 +510,7 @@ emitRegisterNameString(raw_ostream &O, StringRef AltName,
     << "  };\n"
     << "\n";
 
-  O << "  const char *const AsmStrs" << AltName << " =\n";
+  O << "  const char *AsmStrs" << AltName << " =\n";
   StringTable.EmitString(O);
   O << ";\n";
 }
@@ -544,7 +544,7 @@ void AsmWriterEmitter::EmitGetRegisterName(raw_ostream &O) {
     emitRegisterNameString(O, "", Registers);
 
   if (hasAltNames) {
-    O << "  const uint16_t *RegAsmOffset;\n"
+    O << "  const unsigned *RegAsmOffset;\n"
       << "  const char *AsmStrs;\n"
       << "  switch(AltIdx) {\n"
       << "  default: llvm_unreachable(\"Invalid register alt name index!\");\n";
@@ -586,7 +586,7 @@ void AsmWriterEmitter::EmitGetInstructionName(raw_ostream &O) {
   << "  assert(Opcode < " << NumberedInstructions.size()
   << " && \"Invalid instruction number!\");\n"
   << "\n"
-  << "  static const uint16_t InstAsmOffset[] = {";
+  << "  static const unsigned InstAsmOffset[] = {";
   for (unsigned i = 0, e = NumberedInstructions.size(); i != e; ++i) {
     const CodeGenInstruction &Inst = *NumberedInstructions[i];
 
@@ -602,7 +602,7 @@ void AsmWriterEmitter::EmitGetInstructionName(raw_ostream &O) {
   << "  };\n"
   << "\n";
 
-  O << "  const char *const Strs =\n";
+  O << "  const char *Strs =\n";
   StringTable.EmitString(O);
   O << ";\n";