Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 7 Sep 2016 20:36:14 +0000 (UTC)
From:      Dimitry Andric <dim@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   svn commit: r305564 - projects/clang390-import/contrib/llvm/lib/Target/X86
Message-ID:  <201609072036.u87KaEwl063545@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: dim
Date: Wed Sep  7 20:36:13 2016
New Revision: 305564
URL: https://svnweb.freebsd.org/changeset/base/305564

Log:
  Pull in r280837 from upstream llvm trunk (by Wei Mi):
  
    Don't reduce the width of vector mul if the target doesn't support
    SSE2.
  
    The patch is to fix PR30298, which is caused by rL272694. The
    solution is to bail out if the target has no SSE2.
  
    Differential Revision: https://reviews.llvm.org/D24288
  
  This fixes building the multimedia/libx264 port on i386.

Modified:
  projects/clang390-import/contrib/llvm/lib/Target/X86/X86ISelLowering.cpp

Modified: projects/clang390-import/contrib/llvm/lib/Target/X86/X86ISelLowering.cpp
==============================================================================
--- projects/clang390-import/contrib/llvm/lib/Target/X86/X86ISelLowering.cpp	Wed Sep  7 20:09:24 2016	(r305563)
+++ projects/clang390-import/contrib/llvm/lib/Target/X86/X86ISelLowering.cpp	Wed Sep  7 20:36:13 2016	(r305564)
@@ -27516,7 +27516,8 @@ static SDValue reduceVMULWidth(SDNode *N
                                const X86Subtarget &Subtarget) {
   // pmulld is supported since SSE41. It is better to use pmulld
   // instead of pmullw+pmulhw.
-  if (Subtarget.hasSSE41())
+  // pmullw/pmulhw are not supported by SSE.
+  if (Subtarget.hasSSE41() || !Subtarget.hasSSE2())
     return SDValue();
 
   ShrinkMode Mode;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201609072036.u87KaEwl063545>