From 6349380fa412d6082ce4ec017080267643b2188f Mon Sep 17 00:00:00 2001 From: Matthew Simpson Date: Wed, 24 May 2017 16:48:39 +0000 Subject: [PATCH] Revert r291254: [AArch64] Reduce vector insert/extract cost for Falkor The default vector insert/extract cost is more profitable on Falkor than the reduced cost. llvm-svn: 303771 --- llvm/lib/Target/AArch64/AArch64Subtarget.cpp | 1 - .../test/Analysis/CostModel/AArch64/falkor.ll | 26 ------------------- 2 files changed, 27 deletions(-) delete mode 100644 llvm/test/Analysis/CostModel/AArch64/falkor.ll diff --git a/llvm/lib/Target/AArch64/AArch64Subtarget.cpp b/llvm/lib/Target/AArch64/AArch64Subtarget.cpp index b369ee7e4ba2..d3cab1ad3397 100644 --- a/llvm/lib/Target/AArch64/AArch64Subtarget.cpp +++ b/llvm/lib/Target/AArch64/AArch64Subtarget.cpp @@ -90,7 +90,6 @@ void AArch64Subtarget::initializeProperties() { break; case Falkor: MaxInterleaveFactor = 4; - VectorInsertExtractBaseCost = 2; // FIXME: remove this to enable 64-bit SLP if performance looks good. MinVectorRegisterBitWidth = 128; break; diff --git a/llvm/test/Analysis/CostModel/AArch64/falkor.ll b/llvm/test/Analysis/CostModel/AArch64/falkor.ll deleted file mode 100644 index e9563191f077..000000000000 --- a/llvm/test/Analysis/CostModel/AArch64/falkor.ll +++ /dev/null @@ -1,26 +0,0 @@ -; RUN: opt < %s -cost-model -analyze -mcpu=falkor | 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 -}