[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