[PATCH] D48172: [CostModel][AArch64] Add some initial costs for SK_Select and SK_PermuteSingleSrc

[Evandro Menezes via Phabricator via llvm-commits]

evandro added inline comments.


================
Comment at: lib/Target/AArch64/AArch64TargetTransformInfo.cpp:963
+  if (Kind == TTI::SK_Select) {
+    static const CostTblEntry SelectTbl[] = {
+        {ISD::VECTOR_SHUFFLE, MVT::v2i32, 1},
----------------
I'm not sure that the costs of 64 and 128 bits long vectors should be different.


================
Comment at: lib/Target/AArch64/AArch64TargetTransformInfo.cpp:978
+  if (Kind == TTI::SK_PermuteSingleSrc) {
+    static const CostTblEntry PermuteTbl[] = {
+        {ISD::VECTOR_SHUFFLE, MVT::v2i32, 1},
----------------
Ditto.


Repository:
  rL LLVM

https://reviews.llvm.org/D48172