[PATCH] D20762: AArch64: Do not test for CPUs, use SubtargetFeatures

[Matthew Simpson via llvm-commits]

mssimpso added a subscriber: mssimpso.

================
Comment at: lib/Target/AArch64/AArch64Subtarget.cpp:69
@@ +68,3 @@
+    break;
+  case ExynosM1:
+  case Kryo:
----------------
Hi Matthias,

I haven't yet looked at this patch in much detail. But would you mind double checking the Exynos properties before committing this change? I didn't think Exynos and Kryo shared these value. Shouldn't MaxInterleaveFactor = 2 and VectorInsertExtractBaseCost = 3 for Exynos? The Kryo numbers look correct. Perhaps there's a missing "break" here?


Repository:
  rL LLVM

http://reviews.llvm.org/D20762