[PATCH] D36691: [AsmParser] Add DiagnosticString to register classes in tablegen

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 09:16:02 PDT 2017


RKSimon added inline comments.


================
Comment at: llvm/trunk/utils/TableGen/AsmMatcherEmitter.cpp:2274
+  OS << "}\n\n";
+}
+
----------------
@olista01 We're seeing "switch statement contains 'default' but no 'case' labels" warnings (and we build with Werror.....).

Please can you update this so that it just returns MCTargetAsmParser::Match_InvalidOperand if there are no regclass entries?


Repository:
  rL LLVM

https://reviews.llvm.org/D36691





More information about the llvm-commits mailing list