[llvm] r346589 - [CostModel][X86] SK_ExtractSubvector costs must only be tested for vector types (PR39615)

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Nov 10 09:37:53 PST 2018


Author: rksimon
Date: Sat Nov 10 09:37:52 2018
New Revision: 346589

URL: http://llvm.org/viewvc/llvm-project?rev=346589&view=rev
Log:
[CostModel][X86] SK_ExtractSubvector costs must only be tested for vector types  (PR39615)

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=346589&r1=346588&r2=346589&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp Sat Nov 10 09:37:52 2018
@@ -874,7 +874,7 @@ int X86TTIImpl::getShuffleCost(TTI::Shuf
 
   // Subvector extractions are free if they start at beginning of the
   // vector.
-  if (Kind == TTI::SK_ExtractSubvector &&
+  if (Kind == TTI::SK_ExtractSubvector && LT.second.isVector() &&
       ((Index % LT.second.getVectorNumElements()) == 0))
     return 0;
 




More information about the llvm-commits mailing list