[PATCH] Exhaustively test all permutations of triple parts in TripleTest.
Chandler Carruth
chandlerc at gmail.com
Thu Apr 30 10:55:12 PDT 2015
Just a minor tweak on my end.
================
Comment at: include/llvm/ADT/Triple.h:87
@@ -86,1 +86,3 @@
+ // Keep this in sync with the preceding list.
+ static const ArchType ArchTypeMax = kalimba;
enum SubArchType {
----------------
In lots of other parts of LLVM, we add an extra "last" enum, I think I'd prefer that slightly.
http://reviews.llvm.org/D9057
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list