Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3488)

Unified Diff: base/third_party/dmg_fp/vs2013-optimization.patch

Issue 385503002: Fixes for re-enabling more MSVC level 4 warnings: base/third_party/dmg_fp/ edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/third_party/dmg_fp/msvc_warnings.patch ('k') | base/third_party/dmg_fp/win_vs2012.patch » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/third_party/dmg_fp/vs2013-optimization.patch
diff --git a/base/third_party/dmg_fp/vs2013-optimization.patch b/base/third_party/dmg_fp/vs2013-optimization.patch
deleted file mode 100644
index d91b370e51fcce41b76c5010400a377073fb39b4..0000000000000000000000000000000000000000
--- a/base/third_party/dmg_fp/vs2013-optimization.patch
+++ /dev/null
@@ -1,18 +0,0 @@
-Index: base/third_party/dmg_fp/dtoa.cc
-diff --git a/base/third_party/dmg_fp/dtoa.cc b/base/third_party/dmg_fp/dtoa.cc
-index 4eb9f0efd94221b3ab95f84554bbc92f112bf973..b03ccff569f9403eb67a95737b0e19740e56ef33 100644
---- a/base/third_party/dmg_fp/dtoa.cc
-+++ b/base/third_party/dmg_fp/dtoa.cc
-@@ -179,6 +179,12 @@
- * used for input more than STRTOD_DIGLIM digits long (default 40).
- */
-
-+#if defined _MSC_VER && _MSC_VER == 1800
-+// TODO(scottmg): VS2013 RC ICEs on a bunch of functions in this file.
-+// This should be removed after RTM. See http://crbug.com/288948.
-+#pragma optimize("", off)
-+#endif
-+
- #define IEEE_8087
- #define NO_HEX_FP
-
« no previous file with comments | « base/third_party/dmg_fp/msvc_warnings.patch ('k') | base/third_party/dmg_fp/win_vs2012.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698