[PATCH] Exhaustively test all permutations of triple parts in TripleTest.

Douglas Katzman dougk at google.com
Thu Apr 30 11:40:57 PDT 2015


================
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 {
----------------
chandlerc wrote:
> In lots of other parts of LLVM, we add an extra "last" enum, I think I'd prefer that slightly.
done. Thanks for the suggestion!

http://reviews.llvm.org/D9057

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list