Changeset 248472 in webkit

Timestamp:

Aug 9, 2019, 1:33:36 PM (7 years ago)

Author:

pvollan@apple.com

Message:

[Win] Remove compiler workaround for VS2013
​https://bugs.webkit.org/show_bug.cgi?id=200582

Reviewed by Don Olmstead.

A VS2013 compiler workaround can be removed now.

• WebKitQuartzCoreAdditions/API/WebKitQuartzCoreAdditions.cpp:

(DllMain):

Location:

trunk/Source/WebKitLegacy/win

Files:

• ChangeLog (modified) (1 diff)
• WebKitQuartzCoreAdditions/API/WebKitQuartzCoreAdditions.cpp (modified) (1 diff)

trunk/Source/WebKitLegacy/win/ChangeLog

@@ +1 @@
+2019-08-09  Per Arne Vollan  <pvollan@apple.com>
+
+        [Win] Remove compiler workaround for VS2013
+        https://bugs.webkit.org/show_bug.cgi?id=200582
+
+        Reviewed by Don Olmstead.
+
+        A VS2013 compiler workaround can be removed now.
+
+        * WebKitQuartzCoreAdditions/API/WebKitQuartzCoreAdditions.cpp:
+        (DllMain):
+
 2019-08-08  Per Arne Vollan  <pvollan@apple.com>
 

trunk/Source/WebKitLegacy/win/WebKitQuartzCoreAdditions/API/WebKitQuartzCoreAdditions.cpp

@@ -44 +44 @@
     switch (fdwReason) {
     case DLL_PROCESS_ATTACH:
-#if defined(_M_X64) || defined(__x86_64__)
-        // The VS2013 runtime has a bug where it mis-detects AVX-capable processors
-        // if the feature has been disabled in firmware. This causes us to crash
-        // in some of the math functions. For now, we disable those optimizations
-        // because Microsoft is not going to fix the problem in VS2013.
-        // FIXME: http://webkit.org/b/141449: Remove this workaround when we switch to VS2015+.
-        _set_FMA3_enable(0);
-#endif
         instance = hinstDLL;
         break;