Bug 1689990 - Part 16: Move assertions to the top of addNativeToBytecodeEntry(). r=jandem

These assertions should hold even when profiling isn't enabled.

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

js/src/jit/shared/CodeGenerator-shared.cpp

@@ -206,6 +206,10 @@ void CodeGeneratorShared::addOutOfLineCode(OutOfLineCode* code,
 }
 
 bool CodeGeneratorShared::addNativeToBytecodeEntry(const BytecodeSite* site) {
+  MOZ_ASSERT(site);
+  MOZ_ASSERT(site->tree());
+  MOZ_ASSERT(site->pc());
+
   // Skip the table entirely if profiling is not enabled.
   if (!isProfilerInstrumentationEnabled()) {
     return true;
@@ -217,10 +221,6 @@ bool CodeGeneratorShared::addNativeToBytecodeEntry(const BytecodeSite* site) {
     return false;
   }
 
-  MOZ_ASSERT(site);
-  MOZ_ASSERT(site->tree());
-  MOZ_ASSERT(site->pc());
-
   InlineScriptTree* tree = site->tree();
   jsbytecode* pc = site->pc();
   uint32_t nativeOffset = masm.currentOffset();