[PATCH] D26673: [AArch64] Add support for Qualcomm's Falkor CPU

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 13:29:15 PST 2016


mcrosier marked an inline comment as done.
mcrosier added inline comments.


================
Comment at: unittests/Support/TargetParserTest.cpp:545-548
   const char *CPU[] = {"cortex-a35", "cortex-a53", "cortex-a57",
                        "cortex-a72", "cortex-a73", "cyclone",
-                       "exynos-m1",  "exynos-m2",  "kryo",
+                       "exynos-m1",  "exynos-m2",  "falkor", "kryo",
                        "vulcan"};
----------------
MatzeB wrote:
> Maybe reformat this list to have 1 name per line. This makes it friendlier for merging/cherry picking. (I also like the fact that I can easily sort such a list of names alphabetically in my editor).
Sure.  Done in r287031.


https://reviews.llvm.org/D26673





More information about the llvm-commits mailing list