[PATCH] D21785: [RFC]Add unittests to {ARM | AArch64}TargetParser

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 6 02:43:51 PDT 2016


rengolin added inline comments.

================
Comment at: include/llvm/Support/TargetParser.h:152
@@ +151,3 @@
+#include "AArch64TargetParser.def"
+  AAK_LAST
+};
----------------
I think we should leave that discussion for when we change this to a class design, or when we move to table-generated tables. I think it should be fine as is, for now.


http://reviews.llvm.org/D21785





More information about the llvm-commits mailing list