mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-22 02:33:06 +01:00
[CostModel] Templatize EntryCost::Cost to allow custom cost metrics
We currently use an unsigned value for our CostTblEntry and TypeConversionCostTblEntry cost tables which is limiting depending on how the target wishes to handle various CostKinds etc. For instance, targets might wish to store separate instruction count, latency or throughput values etc. On D46276 we have been investigating storing a code snippet to improve latency/throughput cost calculations. There is a slight problem in that template argument deduction was struggling to match the now templatized Costs[] tables in a ArrayRef constructor - I've added helper wrappers for CostTableLookup/ConvertCostTableLookup which avoids us having to update all existing calls with a template hint. Differential Revision: https://reviews.llvm.org/D106351
This commit is contained in:
parent
1c3574eaae
commit
41db4b6343
@ -21,16 +21,19 @@
|
||||
namespace llvm {
|
||||
|
||||
/// Cost Table Entry
|
||||
struct CostTblEntry {
|
||||
template <typename CostType>
|
||||
struct CostTblEntryT {
|
||||
int ISD;
|
||||
MVT::SimpleValueType Type;
|
||||
unsigned Cost;
|
||||
CostType Cost;
|
||||
};
|
||||
using CostTblEntry = CostTblEntryT<unsigned>;
|
||||
|
||||
/// Find in cost table, TypeTy must be comparable to CompareTy by ==
|
||||
inline const CostTblEntry *CostTableLookup(ArrayRef<CostTblEntry> Tbl,
|
||||
int ISD, MVT Ty) {
|
||||
auto I = find_if(Tbl, [=](const CostTblEntry &Entry) {
|
||||
/// Find in cost table.
|
||||
template <class CostType>
|
||||
inline const CostTblEntryT<CostType> *
|
||||
CostTableLookup(ArrayRef<CostTblEntryT<CostType>> Tbl, int ISD, MVT Ty) {
|
||||
auto I = find_if(Tbl, [=](const CostTblEntryT<CostType> &Entry) {
|
||||
return ISD == Entry.ISD && Ty == Entry.Type;
|
||||
});
|
||||
if (I != Tbl.end())
|
||||
@ -40,20 +43,30 @@ inline const CostTblEntry *CostTableLookup(ArrayRef<CostTblEntry> Tbl,
|
||||
return nullptr;
|
||||
}
|
||||
|
||||
template <size_t N, class CostType>
|
||||
inline const CostTblEntryT<CostType> *
|
||||
CostTableLookup(const CostTblEntryT<CostType> (&Table)[N], int ISD, MVT Ty) {
|
||||
// Wrapper to fix template argument deduction failures.
|
||||
return CostTableLookup<CostType>(makeArrayRef(Table), ISD, Ty);
|
||||
}
|
||||
|
||||
/// Type Conversion Cost Table
|
||||
struct TypeConversionCostTblEntry {
|
||||
template <typename CostType>
|
||||
struct TypeConversionCostTblEntryT {
|
||||
int ISD;
|
||||
MVT::SimpleValueType Dst;
|
||||
MVT::SimpleValueType Src;
|
||||
unsigned Cost;
|
||||
CostType Cost;
|
||||
};
|
||||
using TypeConversionCostTblEntry = TypeConversionCostTblEntryT<unsigned>;
|
||||
|
||||
/// Find in type conversion cost table, TypeTy must be comparable to CompareTy
|
||||
/// by ==
|
||||
inline const TypeConversionCostTblEntry *
|
||||
ConvertCostTableLookup(ArrayRef<TypeConversionCostTblEntry> Tbl,
|
||||
/// Find in type conversion cost table.
|
||||
template <class CostType>
|
||||
inline const TypeConversionCostTblEntryT<CostType> *
|
||||
ConvertCostTableLookup(ArrayRef<TypeConversionCostTblEntryT<CostType>> Tbl,
|
||||
int ISD, MVT Dst, MVT Src) {
|
||||
auto I = find_if(Tbl, [=](const TypeConversionCostTblEntry &Entry) {
|
||||
auto I =
|
||||
find_if(Tbl, [=](const TypeConversionCostTblEntryT<CostType> &Entry) {
|
||||
return ISD == Entry.ISD && Src == Entry.Src && Dst == Entry.Dst;
|
||||
});
|
||||
if (I != Tbl.end())
|
||||
@ -63,6 +76,14 @@ ConvertCostTableLookup(ArrayRef<TypeConversionCostTblEntry> Tbl,
|
||||
return nullptr;
|
||||
}
|
||||
|
||||
template <size_t N, class CostType>
|
||||
inline const TypeConversionCostTblEntryT<CostType> *
|
||||
ConvertCostTableLookup(const TypeConversionCostTblEntryT<CostType> (&Table)[N],
|
||||
int ISD, MVT Dst, MVT Src) {
|
||||
// Wrapper to fix template argument deduction failures.
|
||||
return ConvertCostTableLookup<CostType>(makeArrayRef(Table), ISD, Dst, Src);
|
||||
}
|
||||
|
||||
} // namespace llvm
|
||||
|
||||
#endif /* LLVM_CODEGEN_COSTTABLE_H_ */
|
||||
|
Loading…
Reference in New Issue
Block a user