Bug 1459845 - followup: Properly enclose debug-only variable. r=bustage CLOSED TREE
authorTooru Fujisawa <arai_a@mac.com>
Thu, 31 May 2018 15:37:12 +0900
changeset 420674 632399edfe3afd9d6956754cf5cc6296b0968724
parent 420673 05084c58cd39f65c141e39d2d28528be29056e96
child 420675 2d1e3d80489de3ac0ab0fef04f20cee5b117cbac
push id34077
push usernerli@mozilla.com
push dateThu, 31 May 2018 21:51:59 +0000
treeherdermozilla-central@42880a726964 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbustage
bugs1459845
milestone62.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 1459845 - followup: Properly enclose debug-only variable. r=bustage CLOSED TREE
js/src/frontend/BytecodeEmitter.cpp
--- a/js/src/frontend/BytecodeEmitter.cpp
+++ b/js/src/frontend/BytecodeEmitter.cpp
@@ -2136,17 +2136,19 @@ class MOZ_STACK_CLASS IfThenElseEmitter
         if (!bce_->emitJumpTargetAndPatch(jumpAroundThen_))
             return false;
 
         // Clear jumpAroundThen_ offset, to tell emitEnd there was an else part.
         jumpAroundThen_ = JumpList();
 
         // Restore stack depth of the then part.
         bce_->stackDepth = thenDepth_;
+#ifdef DEBUG
         state_ = State::Else;
+#endif
         return true;
     }
 
   public:
     MOZ_MUST_USE bool emitElse() {
         MOZ_ASSERT(state_ == State::ThenElse || state_ == State::Cond);
 
         if (!emitElseInternal())