Bug 1305819 - set SSSE3_FLAGS for x86-64 Windows clang-cl builds; r=glandium
authorNathan Froyd <froydnj@mozilla.com>
Tue, 27 Sep 2016 15:35:39 -0400
changeset 361376 35a6325bf1f0dedccb4d9df1e108e1c485cf8607
parent 361375 9d0a04ae04cec4c320f88157b640f2bb6007677f
child 361377 37ea2043dfb9d012a64480f4ded7fc457b48e532
push id1369
push userjlorenzo@mozilla.com
push dateMon, 27 Feb 2017 14:59:41 +0000
treeherdermozilla-release@d75a1dba431f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersglandium
bugs1305819
milestone52.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1305819 - set SSSE3_FLAGS for x86-64 Windows clang-cl builds; r=glandium 64-bit Windows builds that use clang-cl need -mssse3 specified, just like the 32-bit builds. MozReview-Commit-ID: KAYXYAAw46I
old-configure.in
--- a/old-configure.in
+++ b/old-configure.in
@@ -1080,20 +1080,21 @@ case "$target" in
             if test -z `echo $CXXFLAGS | grep -i [-/]arch:`; then
               CXXFLAGS="$CXXFLAGS -arch:SSE2"
             fi
             SSE_FLAGS="-arch:SSE"
             SSE2_FLAGS="-arch:SSE2"
             dnl MSVC allows the use of intrinsics without any flags
             dnl and doesn't have a separate arch for SSSE3
             SSSE3_FLAGS="-arch:SSE2"
-            dnl clang-cl, however, requires the appropriate flags.
-            if test -n "$CLANG_CL"; then
-                SSSE3_FLAGS="-mssse3"
-            fi
+        fi
+        dnl clang-cl requires appropriate flags to enable SSSE3 support
+        dnl on all architectures.
+        if test -n "$CLANG_CL"; then
+            SSSE3_FLAGS="-mssse3"
         fi
         dnl VS2013+ requires -FS when parallel building by make -jN.
         dnl If nothing, compiler sometimes causes C1041 error.
         CFLAGS="$CFLAGS -FS"
         CXXFLAGS="$CXXFLAGS -FS"
         # khuey says we can safely ignore MSVC warning C4251
         # MSVC warning C4244 (implicit type conversion may lose data) warns
         # and requires workarounds for perfectly valid code.  Also, GCC/clang