Bug 723197 - Remove a workaround of VS2010 PGO bug. r=dbaron
authorMakoto Kato <m_kato@ga2.so-net.ne.jp>
Fri, 10 Apr 2015 17:09:11 +0900
changeset 238502 38291788a6f41b06a75d681af2b11ad2814223ef
parent 238501 3d38764239c2fc52d53b2d0fb5e7e42ed77e4dce
child 238503 0097fbcadabc9b0ce5a2f167a2e7cca633013681
push id58232
push userm_kato@ga2.so-net.ne.jp
push dateFri, 10 Apr 2015 08:10:36 +0000
treeherdermozilla-inbound@38291788a6f4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs723197
milestone40.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 723197 - Remove a workaround of VS2010 PGO bug. r=dbaron
layout/style/nsCSSParser.cpp
--- a/layout/style/nsCSSParser.cpp
+++ b/layout/style/nsCSSParser.cpp
@@ -1603,23 +1603,16 @@ CSSParserImpl::ParseRule(const nsAString
       OUTPUT_ERROR();
     }
   }
 
   ReleaseScanner();
   return rv;
 }
 
-// See Bug 723197
-#ifdef _MSC_VER
-#pragma optimize( "", off )
-#pragma warning( push )
-#pragma warning( disable : 4748 )
-#endif
-
 void
 CSSParserImpl::ParseProperty(const nsCSSProperty aPropID,
                              const nsAString& aPropValue,
                              nsIURI* aSheetURI,
                              nsIURI* aBaseURI,
                              nsIPrincipal* aSheetPrincipal,
                              css::Declaration* aDeclaration,
                              bool* aChanged,
@@ -1745,21 +1738,16 @@ CSSParserImpl::ParseVariable(const nsASt
     *aChanged = true;
   }
 
   mTempData.AssertInitialState();
 
   ReleaseScanner();
 }
 
-#ifdef _MSC_VER
-#pragma warning( pop )
-#pragma optimize( "", on )
-#endif
-
 void
 CSSParserImpl::ParseMediaList(const nsSubstring& aBuffer,
                               nsIURI* aURI, // for error reporting
                               uint32_t aLineNumber, // for error reporting
                               nsMediaList* aMediaList,
                               bool aHTMLMode)
 {
   // XXX Are there cases where the caller wants to keep what it already