Bug 1521491 part 1 - Stop treating JSOP_ENDITER as a jump target op. r=nbp

Differential Revision: https://phabricator.services.mozilla.com/D17112

--HG--
extra : moz-landing-system : lando

js/src/jit/BaselineCompiler.cpp

@@ -4735,9 +4735,6 @@ bool BaselineCodeGen<Handler>::emit_JSOP_ISNOITER() {
 
 template <typename Handler>
 bool BaselineCodeGen<Handler>::emit_JSOP_ENDITER() {
-  if (!emit_JSOP_JUMPTARGET()) {
-    return false;
-  }
   frame.popRegsAndSync(1);
 
   if (!emitNextIC()) {

js/src/vm/BytecodeUtil.h

@@ -356,7 +356,6 @@ static inline bool BytecodeIsJumpTarget(JSOp op) {
     case JSOP_JUMPTARGET:
     case JSOP_LOOPHEAD:
     case JSOP_LOOPENTRY:
-    case JSOP_ENDITER:
     case JSOP_TRY:
       return true;
     default:

js/src/vm/Interpreter.cpp

@@ -2258,7 +2258,6 @@ static MOZ_NEVER_INLINE JS_HAZ_JSNATIVE_CALLER bool Interpret(JSContext* cx,
 
     CASE(JSOP_ENDITER) {
       MOZ_ASSERT(REGS.stackDepth() >= 1);
-      COUNT_COVERAGE();
       CloseIterator(&REGS.sp[-1].toObject());
       REGS.sp--;
     }