Bug 1333645 - Remove the VS2013 MaxILKSize workaround. r=glandium
authorDavid Major <dmajor@mozilla.com>
Thu, 26 Jan 2017 13:28:44 +1300
changeset 466672 b2c4e4641643e39e317e3bbf2c940bfb788a884e
parent 466671 9e55d80cc4a819f1ab122e77f28edbe5d18a2bd6
child 466673 00cbc26bec7fdc29dce591bfed54b30b774d106a
push id42948
push userbmo:gasolin@mozilla.com
push dateThu, 26 Jan 2017 07:49:21 +0000
reviewersglandium
bugs1333645
milestone54.0a1
Bug 1333645 - Remove the VS2013 MaxILKSize workaround. r=glandium MozReview-Commit-ID: 7V5jWL6mkVo
old-configure.in
--- a/old-configure.in
+++ b/old-configure.in
@@ -1120,26 +1120,21 @@ case "$target" in
         dnl optimization/PGO case. I think it's probably a compiler bug,
         dnl but we work around it here.
         PROFILE_USE_CFLAGS="-GL -wd4624 -wd4952"
         dnl XXX: should be -LTCG:PGOPTIMIZE, but that fails on libxul.
         dnl Probably also a compiler bug, but what can you do?
         PROFILE_USE_LDFLAGS="-LTCG:PGUPDATE $cgthreads"
         LDFLAGS="$LDFLAGS -DYNAMICBASE"
         RCFLAGS="-nologo"
-        if test "$CC_VERSION" = "18.00.31101"; then
-            dnl Use MaxILKSize as a workaround for LNK1248 in VS2013update4
-            dnl See https://connect.microsoft.com/VisualStudio/feedback/details/1044914/fatal-error-lnk1248
-            LDFLAGS="$LDFLAGS -MaxILKSize:0x7FF00000"
-        fi
-        dnl Minimum reqiurement of Gecko is VS2010 or later which supports
+        dnl Minimum reqiurement of Gecko is VS2015 or later which supports
         dnl both SSSE3 and SSE4.1.
         HAVE_TOOLCHAIN_SUPPORT_MSSSE3=1
         HAVE_TOOLCHAIN_SUPPORT_MSSE4_1=1
-        dnl allow AVX2 code from VS2012
+        dnl allow AVX2 code from VS2015
         HAVE_X86_AVX2=1
     fi
     AC_DEFINE(WIN32_LEAN_AND_MEAN)
     dnl See http://support.microsoft.com/kb/143208 to use STL
     AC_DEFINE(NOMINMAX)
     BIN_SUFFIX='.exe'
     MOZ_USER_DIR="Mozilla"