mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-23 03:02:36 +01:00
[CodeGenPrepare][NFC] Rename TargetTransformInfo::expandMemCmp -> TargetTransformInfo::enableMemCmpExpansion.
Summary: Right now there are two functions with the same name, one does the work and the other one returns true if expansion is needed. Rename TargetTransformInfo::expandMemCmp to make it more consistent with other members of TargetTransformInfo. Remove the unused Instruction* parameter. Differential Revision: https://reviews.llvm.org/D38165 llvm-svn: 314096
This commit is contained in:
parent
7464cec0af
commit
ee8300aee3
@ -548,7 +548,7 @@ public:
|
||||
bool enableAggressiveInterleaving(bool LoopHasReductions) const;
|
||||
|
||||
/// \brief Enable inline expansion of memcmp
|
||||
bool expandMemCmp(Instruction *I, unsigned &MaxLoadSize) const;
|
||||
bool enableMemCmpExpansion(unsigned &MaxLoadSize) const;
|
||||
|
||||
/// \brief Enable matching of interleaved access groups.
|
||||
bool enableInterleavedAccessVectorization() const;
|
||||
@ -985,7 +985,7 @@ public:
|
||||
unsigned VF) = 0;
|
||||
virtual bool supportsEfficientVectorElementLoadStore() = 0;
|
||||
virtual bool enableAggressiveInterleaving(bool LoopHasReductions) = 0;
|
||||
virtual bool expandMemCmp(Instruction *I, unsigned &MaxLoadSize) = 0;
|
||||
virtual bool enableMemCmpExpansion(unsigned &MaxLoadSize) = 0;
|
||||
virtual bool enableInterleavedAccessVectorization() = 0;
|
||||
virtual bool isFPVectorizationPotentiallyUnsafe() = 0;
|
||||
virtual bool allowsMisalignedMemoryAccesses(LLVMContext &Context,
|
||||
@ -1235,8 +1235,8 @@ public:
|
||||
bool enableAggressiveInterleaving(bool LoopHasReductions) override {
|
||||
return Impl.enableAggressiveInterleaving(LoopHasReductions);
|
||||
}
|
||||
bool expandMemCmp(Instruction *I, unsigned &MaxLoadSize) override {
|
||||
return Impl.expandMemCmp(I, MaxLoadSize);
|
||||
bool enableMemCmpExpansion(unsigned &MaxLoadSize) override {
|
||||
return Impl.enableMemCmpExpansion(MaxLoadSize);
|
||||
}
|
||||
bool enableInterleavedAccessVectorization() override {
|
||||
return Impl.enableInterleavedAccessVectorization();
|
||||
|
@ -290,7 +290,7 @@ public:
|
||||
|
||||
bool enableAggressiveInterleaving(bool LoopHasReductions) { return false; }
|
||||
|
||||
bool expandMemCmp(Instruction *I, unsigned &MaxLoadSize) { return false; }
|
||||
bool enableMemCmpExpansion(unsigned &MaxLoadSize) { return false; }
|
||||
|
||||
bool enableInterleavedAccessVectorization() { return false; }
|
||||
|
||||
|
@ -245,8 +245,8 @@ bool TargetTransformInfo::enableAggressiveInterleaving(bool LoopHasReductions) c
|
||||
return TTIImpl->enableAggressiveInterleaving(LoopHasReductions);
|
||||
}
|
||||
|
||||
bool TargetTransformInfo::expandMemCmp(Instruction *I, unsigned &MaxLoadSize) const {
|
||||
return TTIImpl->expandMemCmp(I, MaxLoadSize);
|
||||
bool TargetTransformInfo::enableMemCmpExpansion(unsigned &MaxLoadSize) const {
|
||||
return TTIImpl->enableMemCmpExpansion(MaxLoadSize);
|
||||
}
|
||||
|
||||
bool TargetTransformInfo::enableInterleavedAccessVectorization() const {
|
||||
|
@ -2315,7 +2315,7 @@ static bool expandMemCmp(CallInst *CI, const TargetTransformInfo *TTI,
|
||||
// TTI call to check if target would like to expand memcmp. Also, get the
|
||||
// MaxLoadSize.
|
||||
unsigned MaxLoadSize;
|
||||
if (!TTI->expandMemCmp(CI, MaxLoadSize))
|
||||
if (!TTI->enableMemCmpExpansion(MaxLoadSize))
|
||||
return false;
|
||||
|
||||
// Early exit from expansion if -Oz.
|
||||
|
@ -215,7 +215,7 @@ bool PPCTTIImpl::enableAggressiveInterleaving(bool LoopHasReductions) {
|
||||
return LoopHasReductions;
|
||||
}
|
||||
|
||||
bool PPCTTIImpl::expandMemCmp(Instruction *I, unsigned &MaxLoadSize) {
|
||||
bool PPCTTIImpl::enableMemCmpExpansion(unsigned &MaxLoadSize) {
|
||||
MaxLoadSize = 8;
|
||||
return true;
|
||||
}
|
||||
|
@ -61,7 +61,7 @@ public:
|
||||
/// @{
|
||||
|
||||
bool enableAggressiveInterleaving(bool LoopHasReductions);
|
||||
bool expandMemCmp(Instruction *I, unsigned &MaxLoadSize);
|
||||
bool enableMemCmpExpansion(unsigned &MaxLoadSize);
|
||||
bool enableInterleavedAccessVectorization();
|
||||
unsigned getNumberOfRegisters(bool Vector);
|
||||
unsigned getRegisterBitWidth(bool Vector) const;
|
||||
|
@ -2536,7 +2536,7 @@ bool X86TTIImpl::areInlineCompatible(const Function *Caller,
|
||||
return (CallerBits & CalleeBits) == CalleeBits;
|
||||
}
|
||||
|
||||
bool X86TTIImpl::expandMemCmp(Instruction *I, unsigned &MaxLoadSize) {
|
||||
bool X86TTIImpl::enableMemCmpExpansion(unsigned &MaxLoadSize) {
|
||||
// TODO: We can increase these based on available vector ops.
|
||||
MaxLoadSize = ST->is64Bit() ? 8 : 4;
|
||||
return true;
|
||||
|
@ -127,7 +127,7 @@ public:
|
||||
bool hasDivRemOp(Type *DataType, bool IsSigned);
|
||||
bool areInlineCompatible(const Function *Caller,
|
||||
const Function *Callee) const;
|
||||
bool expandMemCmp(Instruction *I, unsigned &MaxLoadSize);
|
||||
bool enableMemCmpExpansion(unsigned &MaxLoadSize);
|
||||
bool enableInterleavedAccessVectorization();
|
||||
private:
|
||||
int getGSScalarCost(unsigned Opcode, Type *DataTy, bool VariableMask,
|
||||
|
Loading…
Reference in New Issue
Block a user