summaryrefslogtreecommitdiffstats
path: root/llvm/utils
diff options
context:
space:
mode:
authorJakob Stoklund Olesen <stoklund@2pi.dk>2012-03-15 18:05:54 +0000
committerJakob Stoklund Olesen <stoklund@2pi.dk>2012-03-15 18:05:54 +0000
commite99d26c68b8f1fade5c9f2f15dd4625b8f037208 (patch)
tree2f1ee132ac16794363d5ade4dfd1369f3fe826b1 /llvm/utils
parent327d2f9da545ef9f95405dae423be56286594c58 (diff)
downloadbcm5719-llvm-e99d26c68b8f1fade5c9f2f15dd4625b8f037208.tar.gz
bcm5719-llvm-e99d26c68b8f1fade5c9f2f15dd4625b8f037208.zip
Revert r152105: "Use uint16_t to store indices into string table"
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. llvm-svn: 152816
Diffstat (limited to 'llvm/utils')
-rw-r--r--llvm/utils/TableGen/AsmWriterEmitter.cpp10
1 files changed, 5 insertions, 5 deletions
diff --git a/llvm/utils/TableGen/AsmWriterEmitter.cpp b/llvm/utils/TableGen/AsmWriterEmitter.cpp
index 7671efdde0a..0ef5a93d569 100644
--- a/llvm/utils/TableGen/AsmWriterEmitter.cpp
+++ b/llvm/utils/TableGen/AsmWriterEmitter.cpp
@@ -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";
OpenPOWER on IntegriCloud