[llvm] r250995 - Partially reverted changes from r250686
Elena Demikhovsky via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 21 23:20:29 PDT 2015
Author: delena
Date: Thu Oct 22 01:20:29 2015
New Revision: 250995
URL: http://llvm.org/viewvc/llvm-project?rev=250995&view=rev
Log:
Partially reverted changes from r250686
Clang runtime failure was reported.
Assertion failed: (isExtended() && "Type is not extended!"), function getTypeForEVT
I'll need to add a proper handling for PointerType in masked load/store intrinsics.
Modified:
llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp
Modified: llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=250995&r1=250994&r2=250995&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp Thu Oct 22 01:20:29 2015
@@ -1191,8 +1191,10 @@ int X86TTIImpl::getIntImmCost(Intrinsic:
bool X86TTIImpl::isLegalMaskedLoad(Type *DataTy) {
Type *ScalarTy = DataTy->getScalarType();
- int DataWidth = ScalarTy->isPointerTy() ? DL.getPointerSizeInBits() :
- ScalarTy->getPrimitiveSizeInBits();
+ // TODO: Pointers should also be legal,
+ // but it requires additional support in composing intrinsics name.
+ // getPrimitiveSizeInBits() returns 0 for PointerType
+ int DataWidth = ScalarTy->getPrimitiveSizeInBits();
return (DataWidth >= 32 && ST->hasAVX2());
}
More information about the llvm-commits
mailing list