[PATCH] D17379: [AArch64] Reduce vector insert/extract cost for Kryo

[Matthew Simpson via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL261237: [AArch64] Reduce vector insert/extract cost for Kryo (authored by mssimpso).

Changed prior to commit:
  http://reviews.llvm.org/D17379?vs=48312&id=48357#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D17379

Files:
  llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
  llvm/trunk/test/Analysis/CostModel/AArch64/kryo.ll

Index: llvm/trunk/test/Analysis/CostModel/AArch64/kryo.ll
===================================================================
--- llvm/trunk/test/Analysis/CostModel/AArch64/kryo.ll
+++ llvm/trunk/test/Analysis/CostModel/AArch64/kryo.ll
@@ -0,0 +1,26 @@
+; RUN: opt < %s -cost-model -analyze -mcpu=kryo | FileCheck %s
+
+target datalayout = "e-m:e-i64:64-i128:128-n32:64-S128"
+target triple = "aarch64--linux-gnu"
+
+; CHECK-LABEL: vectorInstrCost
+define void @vectorInstrCost() {
+
+    ; Vector extracts - extracting the first element should have a zero cost;
+    ; all other elements should have a cost of two.
+    ;
+    ; CHECK: cost of 0 {{.*}} extractelement <2 x i64> undef, i32 0
+    ; CHECK: cost of 2 {{.*}} extractelement <2 x i64> undef, i32 1
+    %t1 = extractelement <2 x i64> undef, i32 0
+    %t2 = extractelement <2 x i64> undef, i32 1
+
+    ; Vector inserts - inserting the first element should have a zero cost; all
+    ; other elements should have a cost of two.
+    ;
+    ; CHECK: cost of 0 {{.*}} insertelement <2 x i64> undef, i64 undef, i32 0
+    ; CHECK: cost of 2 {{.*}} insertelement <2 x i64> undef, i64 undef, i32 1
+    %t3 = insertelement <2 x i64> undef, i64 undef, i32 0
+    %t4 = insertelement <2 x i64> undef, i64 undef, i32 1
+
+    ret void
+}
Index: llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
===================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
+++ llvm/trunk/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
@@ -313,6 +313,8 @@
   }
 
   // All other insert/extracts cost this much.
+  if (ST->isKryo())
+    return 2;
   return 3;
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17379.48357.patch
Type: text/x-patch
Size: 1688 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160218/89777c28/attachment.bin>