[PATCH] D47150: [Clang Tablegen] Add default case to getModifierName()

2018-05-21 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. Please instead add an `llvm_unreachable` outside the switch, so that we still get warnings on missing switch cases if an enumerator is added. https://reviews.llvm.org/D47150 ___ cfe-commits mailing list

[PATCH] D47150: [Clang Tablegen] Add default case to getModifierName()

2018-05-21 Thread Mark Searles via Phabricator via cfe-commits
msearles created this revision. msearles added reviewers: b-sumner, kzhuravl, EricWF, rjmccall. Herald added subscribers: ioeric, ilya-biryukov. Fix internal build failure: ../../../ClangDiagnosticsEmitter.cpp -o ClangDiagnosticsEmitter.o ../../../ClangDiagnosticsEmitter.cpp: In function