[TableGen] Allow mnemonics aliases with uppercase

[paperchalice]

Summary

Improves rG3cf7efec986d. Let applyMnemonicAliases work correctly when mnemonic and mnemonic alias contain uppercase characters.

[Paul-C-Anagnostopoulos]

This looks straightforward. Did you build all the targets and make sure the .inc files didn't change?

[paperchalice]

All inc under Target not are not changed

[paperchalice]

change directory layout to monorepo

[Paul-C-Anagnostopoulos]

Looks good to me.

[paperchalice]

Sorry, I don't have commit access. Thanks in advance if someone can commit the change. @Paul-C-Anagnostopoulos

[Paul-C-Anagnostopoulos]

I will push this revision in the next few days.

[Paul-C-Anagnostopoulos]

I am pushing this revision today.

llvm/test/TableGen/MixedCasedMnemonic.td

 // RUN: llvm-tblgen -gen-asm-matcher -I %p/../../include %s | FileCheck %s --check-prefix=MATCHER
 // RUN: llvm-tblgen -gen-asm-writer -I %p/../../include %s | FileCheck %s --check-prefix=WRITER
+// RUN: llvm-tblgen -gen-asm-matcher -I %p/../../include %s | FileCheck %s --check-prefix=ALIAS
 
 // Check that an instruction that uses mixed upper/lower case in its mnemonic
 // is printed as-is, and is parsed in its "canonicalized" lowercase form.

   let AsmString = "aInst";
 }
 
+def :MnemonicAlias<"Insta", "aInst">;
+def :MnemonicAlias<"InstB", "BInst">;
+
 // Check that the matcher lower()s the mnemonics it matches.
 // MATCHER: static const char *const MnemonicTable =
 // MATCHER-NEXT: "\005ainst\005binst";

 // WRITER-NEXT:   "aInst\0"
 // WRITER-NEXT: };
 
+// ALIAS: static void applyMnemonicAliases(StringRef &Mnemonic, const FeatureBitset &Features, unsigned VariantID) {
+// ALIAS-NEXT  switch (VariantID) {
+// ALIAS-NEXT  case 0:
+// ALIAS-NEXT      switch (Mnemonic.size()) {
+// ALIAS-NEXT      default: break;
+// ALIAS-NEXT      case 5:   // 2 strings to match.
+// ALIAS-NEXT        if (memcmp(Mnemonic.data()+0, "inst", 4) != 0)
+// ALIAS-NEXT          break;
+// ALIAS-NEXT        switch (Mnemonic[4]) {
+// ALIAS-NEXT        default: break;
+// ALIAS-NEXT        case 'a':   // 1 string to match.
+// ALIAS-NEXT          Mnemonic = "ainst";   // "insta"
+// ALIAS-NEXT          return;
+// ALIAS-NEXT        case 'b':   // 1 string to match.
+// ALIAS-NEXT          Mnemonic = "binst";   // "instb"
+// ALIAS-NEXT          return;
+

llvm/utils/TableGen/AsmMatcherEmitter.cpp

     StringRef AsmVariantName = R->getValueAsString("AsmVariantName");
     if (AsmVariantName != AsmParserVariantName)
       continue;
-    AliasesFromMnemonic[std::string(R->getValueAsString("FromMnemonic"))]
+    AliasesFromMnemonic[R->getValueAsString("FromMnemonic").lower()]
         .push_back(R);
   }
   if (AliasesFromMnemonic.empty())

         MatchCode += "else ";
       MatchCode += "if (" + FeatureMask + ")\n";
       MatchCode += "  Mnemonic = \"";
-      MatchCode += R->getValueAsString("ToMnemonic");
+      MatchCode += R->getValueAsString("ToMnemonic").lower();
       MatchCode += "\";\n";
     }
 

       if (!MatchCode.empty())
         MatchCode += "else\n  ";
       MatchCode += "Mnemonic = \"";
-      MatchCode += R->getValueAsString("ToMnemonic");
+      MatchCode += R->getValueAsString("ToMnemonic").lower();
       MatchCode += "\";\n";
     }