[PATCH] D14994: Enable MatchRegisterName to match register altnames

Colin LeMahieu via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 12:19:02 PST 2015


colinl added a comment.

Is there a reason to not emit all AltNames as register matches?  This way we aren't putting a requirement on the number of altnames for a register.

Maybe in emitMatchRegisterName:

      Matches.emplace_back(Reg.TheDef->getValueAsString("AsmName"),
                           "return " + utostr(Reg.EnumValue) + ";");
  +    for (auto &I: Reg.TheDef->getValueAsListOfStrings("AltNames"))
  +      Matches.emplace_back(*I, "return " + utostr(Reg.EnumValue) + ";");


http://reviews.llvm.org/D14994





More information about the llvm-commits mailing list