mirror of
https://github.com/capstone-engine/llvm-capstone.git
synced 2024-11-24 06:10:12 +00:00
Fixes ARM LNT bot from SLP change in O3
This patch fixes the multiple breakages on ARM test-suite after the SLP vectorizer was introduced by default on O3. The problem was an illegal vector type on ARMTTI::getCmpSelInstrCost() <3 x i1> which is not simple. The guard protects this code from breaking (cause of the problems) but doesn't fix the issue that is generating the odd vector in the first place, which also needs to be investigated. llvm-svn: 187658
This commit is contained in:
parent
87dc0bc078
commit
0178a25fc5
@ -411,12 +411,14 @@ unsigned ARMTTI::getCmpSelInstrCost(unsigned Opcode, Type *ValTy,
|
||||
|
||||
EVT SelCondTy = TLI->getValueType(CondTy);
|
||||
EVT SelValTy = TLI->getValueType(ValTy);
|
||||
int Idx = ConvertCostTableLookup<MVT>(NEONVectorSelectTbl,
|
||||
array_lengthof(NEONVectorSelectTbl),
|
||||
ISD, SelCondTy.getSimpleVT(),
|
||||
SelValTy.getSimpleVT());
|
||||
if (Idx != -1)
|
||||
return NEONVectorSelectTbl[Idx].Cost;
|
||||
if (SelCondTy.isSimple() && SelValTy.isSimple()) {
|
||||
int Idx = ConvertCostTableLookup<MVT>(NEONVectorSelectTbl,
|
||||
array_lengthof(NEONVectorSelectTbl),
|
||||
ISD, SelCondTy.getSimpleVT(),
|
||||
SelValTy.getSimpleVT());
|
||||
if (Idx != -1)
|
||||
return NEONVectorSelectTbl[Idx].Cost;
|
||||
}
|
||||
|
||||
std::pair<unsigned, MVT> LT = TLI->getTypeLegalizationCost(ValTy);
|
||||
return LT.first;
|
||||
|
@ -63,5 +63,13 @@ define void @casts() {
|
||||
; CHECK: cost of 1 {{.*}} select
|
||||
%v19 = select <2 x i1> undef, <2 x double> undef, <2 x double> undef
|
||||
|
||||
; odd vectors get legalized and should have similar costs
|
||||
; CHECK: cost of 1 {{.*}} select
|
||||
%v20 = select <1 x i1> undef, <1 x i32> undef, <1 x i32> undef
|
||||
; CHECK: cost of 1 {{.*}} select
|
||||
%v21 = select <3 x i1> undef, <3 x float> undef, <3 x float> undef
|
||||
; CHECK: cost of 4 {{.*}} select
|
||||
%v22 = select <5 x i1> undef, <5 x double> undef, <5 x double> undef
|
||||
|
||||
ret void
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user