Bug 1289990 - Remove VS2013 workaround for TestTuple.cpp crash. r=froydnj
authorChris Peterson <cpeterson@mozilla.com>
Wed, 27 Jul 2016 23:13:08 -0700
changeset 349453 bfb76ef187d9793edbc16aab9c42555a4b7c347f
parent 349452 43490acc4f626aedde721d48af259bbd0f516503
child 349454 d0cb076a4e736e89f8393ba5658148ae5be4171d
push id1230
push userjlund@mozilla.com
push dateMon, 31 Oct 2016 18:13:35 +0000
treeherdermozilla-release@5e06e3766db2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj
bugs1289990
milestone50.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 1289990 - Remove VS2013 workaround for TestTuple.cpp crash. r=froydnj
mfbt/tests/TestTuple.cpp
--- a/mfbt/tests/TestTuple.cpp
+++ b/mfbt/tests/TestTuple.cpp
@@ -22,22 +22,16 @@ using mozilla::MakeUnique;
 using mozilla::Move;
 using mozilla::Pair;
 using mozilla::Tie;
 using mozilla::Tuple;
 using mozilla::UniquePtr;
 using mozilla::Unused;
 using std::pair;
 
-#if (_MSC_VER == 1800)
-// Optimizations in VS2013 on Windows 10 cause an internal compiler error
-// with this file.
-#pragma optimize("", off)
-#endif
-
 #define CHECK(c) \
   do { \
     bool cond = !!(c); \
     MOZ_RELEASE_ASSERT(cond, "Failed assertion: " #c); \
   } while (false)
 
 // The second argument is the expected type. It's variadic to allow the
 // type to contain commas.