From 477fba4f54b4fa910364ca4c84c4d6ef37ab44f1 Mon Sep 17 00:00:00 2001
From: Chris Lattner <sabre@nondot.org>
Date: Sat, 30 Oct 2010 18:48:18 +0000
Subject: [PATCH] emit the mnemonic aliases in their own helper function
 instead of inline into MatchInstructionImpl.

llvm-svn: 117826
---
 llvm/utils/TableGen/AsmMatcherEmitter.cpp | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/llvm/utils/TableGen/AsmMatcherEmitter.cpp b/llvm/utils/TableGen/AsmMatcherEmitter.cpp
index 99a32d8a000a..93d3ac304ea3 100644
--- a/llvm/utils/TableGen/AsmMatcherEmitter.cpp
+++ b/llvm/utils/TableGen/AsmMatcherEmitter.cpp
@@ -1515,23 +1515,28 @@ static void EmitComputeAvailableFeatures(CodeGenTarget &Target,
 }
 
 /// EmitMnemonicAliases - If the target has any MnemonicAlias<> definitions,
-/// emit them.
-static void EmitMnemonicAliases(raw_ostream &OS) {
+/// emit a function for them and return true, otherwise return false.
+static bool EmitMnemonicAliases(raw_ostream &OS) {
+  OS << "static void ApplyMnemonicAliases(StringRef &Mnemonic, "
+  "unsigned Features) {\n";
+  
   std::vector<Record*> Aliases =
     Records.getAllDerivedDefinitions("MnemonicAlias");
-  if (Aliases.empty()) return;
+  if (Aliases.empty()) return false;
 
-  OS << "  // Process all MnemonicAliases to remap the mnemonic.\n";
   std::vector<StringMatcher::StringPair> Cases;
   for (unsigned i = 0, e = Aliases.size(); i != e; ++i) {
     Record *R = Aliases[i];
     Cases.push_back(std::make_pair(R->getValueAsString("FromMnemonic"),
                                    "Mnemonic = \"" +
                                    R->getValueAsString("ToMnemonic") +
-                                   "\"; break;"));
+                                   "\"; return;"));
   }
   
   StringMatcher("Mnemonic", Cases, OS).Emit();
+  OS << "}\n";
+  
+  return true;
 }
 
 void AsmMatcherEmitter::run(raw_ostream &OS) {
@@ -1617,6 +1622,9 @@ void AsmMatcherEmitter::run(raw_ostream &OS) {
   OS << "\n#ifdef GET_MATCHER_IMPLEMENTATION\n";
   OS << "#undef GET_MATCHER_IMPLEMENTATION\n\n";
 
+  // Generate the function that remaps for mnemonic aliases.
+  bool HasMnemonicAliases = EmitMnemonicAliases(OS);
+  
   // Generate the unified function to convert operands into an MCInst.
   EmitConvertToMCInst(Target, Info.Instructions, OS);
 
@@ -1725,7 +1733,10 @@ void AsmMatcherEmitter::run(raw_ostream &OS) {
   OS << "  StringRef Mnemonic = ((" << Target.getName()
      << "Operand*)Operands[0])->getToken();\n\n";
 
-  EmitMnemonicAliases(OS);
+  if (HasMnemonicAliases) {
+    OS << "  // Process all MnemonicAliases to remap the mnemonic.\n";
+    OS << "  ApplyMnemonicAliases(Mnemonic, AvailableFeatures);\n\n";
+  }
   
   // Emit code to compute the class list for this operand vector.
   OS << "  // Eliminate obvious mismatches.\n";