[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
//===-- RISCVTargetTransformInfo.cpp - RISC-V specific TTI ----------------===//
|
|
|
|
//
|
|
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
|
|
|
#include "RISCVTargetTransformInfo.h"
|
2021-01-15 03:44:02 +08:00
|
|
|
#include "MCTargetDesc/RISCVMatInt.h"
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
#include "llvm/Analysis/TargetTransformInfo.h"
|
|
|
|
#include "llvm/CodeGen/BasicTTIImpl.h"
|
|
|
|
#include "llvm/CodeGen/TargetLowering.h"
|
|
|
|
using namespace llvm;
|
|
|
|
|
|
|
|
#define DEBUG_TYPE "riscvtti"
|
|
|
|
|
2020-04-28 21:11:27 +08:00
|
|
|
int RISCVTTIImpl::getIntImmCost(const APInt &Imm, Type *Ty,
|
|
|
|
TTI::TargetCostKind CostKind) {
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
assert(Ty->isIntegerTy() &&
|
|
|
|
"getIntImmCost can only estimate cost of materialising integers");
|
|
|
|
|
|
|
|
// We have a Zero register, so 0 is always free.
|
|
|
|
if (Imm == 0)
|
|
|
|
return TTI::TCC_Free;
|
|
|
|
|
|
|
|
// Otherwise, we check how many instructions it will take to materialise.
|
|
|
|
const DataLayout &DL = getDataLayout();
|
|
|
|
return RISCVMatInt::getIntMatCost(Imm, DL.getTypeSizeInBits(Ty),
|
|
|
|
getST()->is64Bit());
|
|
|
|
}
|
|
|
|
|
2020-09-22 19:54:10 +08:00
|
|
|
int RISCVTTIImpl::getIntImmCostInst(unsigned Opcode, unsigned Idx,
|
|
|
|
const APInt &Imm, Type *Ty,
|
|
|
|
TTI::TargetCostKind CostKind,
|
|
|
|
Instruction *Inst) {
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
assert(Ty->isIntegerTy() &&
|
|
|
|
"getIntImmCost can only estimate cost of materialising integers");
|
|
|
|
|
|
|
|
// We have a Zero register, so 0 is always free.
|
|
|
|
if (Imm == 0)
|
|
|
|
return TTI::TCC_Free;
|
|
|
|
|
|
|
|
// Some instructions in RISC-V can take a 12-bit immediate. Some of these are
|
|
|
|
// commutative, in others the immediate comes from a specific argument index.
|
|
|
|
bool Takes12BitImm = false;
|
|
|
|
unsigned ImmArgIdx = ~0U;
|
|
|
|
|
|
|
|
switch (Opcode) {
|
|
|
|
case Instruction::GetElementPtr:
|
|
|
|
// Never hoist any arguments to a GetElementPtr. CodeGenPrepare will
|
|
|
|
// split up large offsets in GEP into better parts than ConstantHoisting
|
|
|
|
// can.
|
|
|
|
return TTI::TCC_Free;
|
|
|
|
case Instruction::Add:
|
|
|
|
case Instruction::And:
|
|
|
|
case Instruction::Or:
|
|
|
|
case Instruction::Xor:
|
|
|
|
case Instruction::Mul:
|
|
|
|
Takes12BitImm = true;
|
|
|
|
break;
|
|
|
|
case Instruction::Sub:
|
|
|
|
case Instruction::Shl:
|
|
|
|
case Instruction::LShr:
|
|
|
|
case Instruction::AShr:
|
|
|
|
Takes12BitImm = true;
|
|
|
|
ImmArgIdx = 1;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (Takes12BitImm) {
|
|
|
|
// Check immediate is the correct argument...
|
|
|
|
if (Instruction::isCommutative(Opcode) || Idx == ImmArgIdx) {
|
|
|
|
// ... and fits into the 12-bit immediate.
|
2019-07-09 18:56:18 +08:00
|
|
|
if (Imm.getMinSignedBits() <= 64 &&
|
|
|
|
getTLI()->isLegalAddImmediate(Imm.getSExtValue())) {
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
return TTI::TCC_Free;
|
2019-07-09 18:56:18 +08:00
|
|
|
}
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// Otherwise, use the full materialisation cost.
|
2020-04-28 21:11:27 +08:00
|
|
|
return getIntImmCost(Imm, Ty, CostKind);
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
// By default, prevent hoisting.
|
|
|
|
return TTI::TCC_Free;
|
|
|
|
}
|
|
|
|
|
2019-12-12 03:54:58 +08:00
|
|
|
int RISCVTTIImpl::getIntImmCostIntrin(Intrinsic::ID IID, unsigned Idx,
|
2020-04-28 21:11:27 +08:00
|
|
|
const APInt &Imm, Type *Ty,
|
|
|
|
TTI::TargetCostKind CostKind) {
|
[RISCV] Add RISCV-specific TargetTransformInfo
Summary:
LLVM Allows Targets to provide information that guides optimisations
made to LLVM IR. This is done with callbacks on a TargetTransformInfo object.
This patch adds a TargetTransformInfo class for RISC-V. This will allow us to
implement RISC-V specific callbacks as they become necessary.
This commit also adds the getIntImmCost callbacks, and tests them with a simple
constant hoisting test. Our immediate costs are on the conservative side, for
the moment, but we prevent hoisting in most circumstances anyway.
Previous review was on D63007
Reviewers: asb, luismarques
Reviewed By: asb
Subscribers: ributzka, MaskRay, llvm-commits, Jim, benna, psnobl, jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook, johnrusso, rbar, hiraditya, mgorny
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D63433
llvm-svn: 364046
2019-06-21 21:36:09 +08:00
|
|
|
// Prevent hoisting in unknown cases.
|
|
|
|
return TTI::TCC_Free;
|
|
|
|
}
|
2021-01-28 23:31:27 +08:00
|
|
|
|
|
|
|
bool RISCVTTIImpl::shouldExpandReduction(const IntrinsicInst *II) const {
|
|
|
|
// Currently, the ExpandReductions pass can't expand scalable-vector
|
|
|
|
// reductions, but we still request expansion as RVV doesn't support certain
|
|
|
|
// reductions and the SelectionDAG can't legalize them either.
|
|
|
|
switch (II->getIntrinsicID()) {
|
|
|
|
default:
|
|
|
|
return false;
|
2021-02-02 22:34:51 +08:00
|
|
|
// These reductions have no equivalent in RVV
|
2021-01-28 23:31:27 +08:00
|
|
|
case Intrinsic::vector_reduce_mul:
|
|
|
|
case Intrinsic::vector_reduce_fmul:
|
2021-02-02 22:34:51 +08:00
|
|
|
// The fmin and fmax intrinsics are not currently supported due to a
|
|
|
|
// discrepancy between the LLVM semantics and the RVV 0.10 ISA behaviour with
|
|
|
|
// regards to signaling NaNs: the vector fmin/fmax reduction intrinsics match
|
|
|
|
// the behaviour minnum/maxnum intrinsics, whereas the vfredmin/vfredmax
|
|
|
|
// instructions match the vfmin/vfmax instructions which match the equivalent
|
|
|
|
// scalar fmin/fmax instructions as defined in 2.2 F/D/Q extension (see
|
|
|
|
// https://bugs.llvm.org/show_bug.cgi?id=27363).
|
|
|
|
// This behaviour is likely fixed in version 2.3 of the RISC-V F/D/Q
|
|
|
|
// extension, where fmin/fmax behave like minnum/maxnum, but until then the
|
|
|
|
// intrinsics are left unsupported.
|
2021-01-28 23:31:27 +08:00
|
|
|
case Intrinsic::vector_reduce_fmax:
|
|
|
|
case Intrinsic::vector_reduce_fmin:
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
}
|