diff options
author | David L. Jones <dlj@google.com> | 2019-05-01 05:01:03 +0000 |
---|---|---|
committer | David L. Jones <dlj@google.com> | 2019-05-01 05:01:03 +0000 |
commit | fccb505f0f59d94b0c46f2dd28fd8893ca45899d (patch) | |
tree | 02c3fe9f18a61c119d354df34c615c247e839912 /llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp | |
parent | 9502e40c9089c84943c9a16fb75c6fa07032786a (diff) | |
download | bcm5719-llvm-fccb505f0f59d94b0c46f2dd28fd8893ca45899d.tar.gz bcm5719-llvm-fccb505f0f59d94b0c46f2dd28fd8893ca45899d.zip |
Revert "[llvm] r359313 - [PowerPC] Update P9 vector costs for insert/extract element"
This causes segfaults during optimized builds. More details, including a reproducer, are on the llvm-commits thread for r359313.
llvm-svn: 359648
Diffstat (limited to 'llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp')
-rw-r--r-- | llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp | 29 |
1 files changed, 0 insertions, 29 deletions
diff --git a/llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp b/llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp index d47674fcfac..fdb2a38419d 100644 --- a/llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp +++ b/llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp @@ -413,35 +413,6 @@ int PPCTTIImpl::getVectorInstrCost(unsigned Opcode, Type *Val, unsigned Index) { return 0; return Cost; - - } else if (Val->getScalarType()->isIntegerTy() && Index != -1U) { - if (ST->hasP9Altivec()) { - if (ISD == ISD::INSERT_VECTOR_ELT) - // A move-to VSR and a permute/insert. Assume vector operation cost - // for both (cost will be 2x on P9). - return vectorCostAdjustment(2, Opcode, Val, nullptr); - - // It's an extract. Maybe we can do a cheap move-from VSR. - unsigned EltSize = Val->getScalarSizeInBits(); - if (EltSize == 64) { - unsigned MfvsrdIndex = ST->isLittleEndian() ? 1 : 0; - if (Index == MfvsrdIndex) - return 1; - } else if (EltSize == 32) { - unsigned MfvsrwzIndex = ST->isLittleEndian() ? 2 : 1; - if (Index == MfvsrwzIndex) - return 1; - } - - // We need a vector extract (or mfvsrld). Assume vector operation cost. - // The cost of the load constant for a vector extract is disregarded - // (invariant, easily schedulable). - return vectorCostAdjustment(1, Opcode, Val, nullptr); - - } else if (ST->hasDirectMove()) - // Assume permute has standard cost. - // Assume move-to/move-from VSR have 2x standard cost. - return 3; } // Estimated cost of a load-hit-store delay. This was obtained |