Bug 1608509 - Part 5: Rename baseline emit methods. r=jandem.

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

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

js/src/frontend/BytecodeEmitter.cpp

@@ -8523,8 +8523,8 @@ bool BytecodeEmitter::emitArray(ParseNode* arrayHead, uint32_t count,
   // Array literal's length is limited to NELEMENTS_LIMIT in parser.
   static_assert(NativeObject::MAX_DENSE_ELEMENTS_COUNT <= INT32_MAX,
                 "array literals' maximum length must not exceed limits "
-                "required by BaselineCompiler::emit_JSOP_NEWARRAY, "
-                "BaselineCompiler::emit_JSOP_INITELEM_ARRAY, "
+                "required by BaselineCompiler::emit_NewArray, "
+                "BaselineCompiler::emit_InitElemArray, "
                 "and DoSetElemFallback's handling of JSOP_INITELEM_ARRAY");
   MOZ_ASSERT(count >= nspread);
   MOZ_ASSERT(count <= NativeObject::MAX_DENSE_ELEMENTS_COUNT,

js/src/jit/BaselineCodeGen.h

@@ -187,7 +187,7 @@ class BaselineCodeGen {
   MOZ_MUST_USE bool emitTraceLoggerResume(Register script,
                                           AllocatableGeneralRegisterSet& regs);
 
-#define EMIT_OP(op, ...) bool emit_##op();
+#define EMIT_OP(op, op_camel, ...) bool emit_##op_camel();
   FOR_EACH_OPCODE(EMIT_OP)
 #undef EMIT_OP
 

js/src/jit/IonBuilder.cpp

@@ -10580,9 +10580,9 @@ AbortReasonOr<Ok> IonBuilder::jsop_rest() {
     }
   }
 
-  // The array's length is incorrectly 0 now, from the template object
-  // created by BaselineCompiler::emit_JSOP_REST() before the actual argument
-  // count was known. Set the correct length now that we know that count.
+  // The array's length is incorrectly 0 now, from the template object created
+  // by BaselineCompiler::emit_Rest() before the actual argument count was
+  // known. Set the correct length now that we know that count.
   MSetArrayLength* length = MSetArrayLength::New(alloc(), elements, index);
   current->add(length);