[PATCH] D37724: [ARM] Add more CPUs to host detection

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: lib/Support/Host.cpp:211
             .Case("0x205", "kryo")
+            .Case("0x211", "kryo")
+            .Case("0x800", "cortex-a73")
----------------
kristof.beyls wrote:
> The summary of this patch states returning cortex-a73 for second-generation Kryo, but this patch also introduces returning "kryo" for "0x211". Just checking if this was intentional.
Yes, intentional; 0x211 is first-generation Kryo.


================
Comment at: unittests/Support/Host.cpp:108-110
+  EXPECT_EQ(sys::detail::getHostCPUNameForARM("CPU implementer : 0x51\n"
+                                              "CPU part        : 0x800"),
+            "cortex-a73");
----------------
kristof.beyls wrote:
> I don't have easy access to a Cortex-A73 system at the moment, but I'd expect the CPU implementer for a Cortex-A73 be 0x41, the same as for other Arm-designed cores.
> Also, according to http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.100048_0002_05_en/jfa1406793219362.html, the part number of Cortex-A73 is 0xD09, not 0x800.
Kryo second-generation is not a Cortex-A73. It's just close enough from the compiler's perspective that it isn't worth distinguishing here.  (getHostCPUNameForARM already has separate detection code for actual A73 CPUs.)


Repository:
  rL LLVM

https://reviews.llvm.org/D37724