[MERGE #6199 @MikeHolman] only tell OOP JIT about modules that have initialized localExportSlots

Merge pull request #6199 from MikeHolman:oopjitmoduleinit

This leads to assert where we add the localExportSlots addr to dictionary when it is null and then try to add again when we actually initialize it (and item is unexpectedly already in dictionary). OOP JIT doesn't need to know about them until they are initialized.

OS: 22218538

Author: MikeHolman

lib/Runtime/Base/ScriptContext.cpp

@@ -5432,14 +5432,20 @@ ScriptContext::GetJitFuncRangeCache()
             JITManager::HandleServerCallResult(hr, RemoteCallType::StateUpdate);
         }
 
-        if (this->GetLibrary()->GetModuleRecordList())
-        {
-            this->GetLibrary()->GetModuleRecordList()->Map([this](int start, SourceTextModuleRecord* moduleRecord) {
-                HRESULT hr = JITManager::GetJITManager()->AddModuleRecordInfo(
-                    m_remoteScriptContextAddr,
-                    moduleRecord->GetModuleId(),
-                    (intptr_t)moduleRecord->GetLocalExportSlots());
-                JITManager::HandleServerCallResult(hr, RemoteCallType::StateUpdate);
+        ModuleRecordList* moduleRecordList = this->GetLibrary()->GetModuleRecordList();
+        if (moduleRecordList)
+        {
+            moduleRecordList->Map([this](int start, SourceTextModuleRecord* moduleRecord) {
+                intptr_t exportSlotsAddr = (intptr_t)moduleRecord->GetLocalExportSlots();
+                // only add modules which have initialized localExportSlots
+                if (exportSlotsAddr)
+                {
+                    HRESULT hr = JITManager::GetJITManager()->AddModuleRecordInfo(
+                        m_remoteScriptContextAddr,
+                        moduleRecord->GetModuleId(),
+                        exportSlotsAddr);
+                    JITManager::HandleServerCallResult(hr, RemoteCallType::StateUpdate);
+                }
             });
         }
     }