Move common runtime includes to runtime_shared.js. NFC (#23269)

This reduces duplication between preamble.js and preamble_minimal.js.

Even though this is completely NFC there were some code size changes due
to the ordering of the code.

Author: sbc100

src/preamble.js

@@ -44,18 +44,6 @@ if (typeof WebAssembly != 'object') {
 }
 #endif
 
-#if SAFE_HEAP
-#include "runtime_safe_heap.js"
-#endif
-
-#if SHARED_MEMORY && ALLOW_MEMORY_GROWTH
-#include "growableHeap.js"
-#endif
-
-#if USE_ASAN
-#include "runtime_asan.js"
-#endif
-
 #if SUPPORT_BASE64_EMBEDDING || FORCE_FILESYSTEM
 #include "base64Utils.js"
 #endif
@@ -150,12 +138,16 @@ var HEAP,
 var HEAP_DATA_VIEW;
 #endif
 
-#include "runtime_shared.js"
+var runtimeInitialized = false;
 
-#if PTHREADS
-#include "runtime_pthread.js"
+#if EXIT_RUNTIME
+var runtimeExited = false;
 #endif
 
+#include "URIUtils.js"
+
+#include "runtime_shared.js"
+
 #if ASSERTIONS
 assert(!Module['STACK_SIZE'], 'STACK_SIZE can no longer be set at runtime.  Use -sSTACK_SIZE at link time')
 #endif
@@ -174,8 +166,6 @@ assert(!Module['wasmMemory'], 'Use of `wasmMemory` detected.  Use -sIMPORTED_MEM
 assert(!Module['INITIAL_MEMORY'], 'Detected runtime INITIAL_MEMORY setting.  Use -sIMPORTED_MEMORY to define wasmMemory dynamically');
 #endif // !IMPORTED_MEMORY && ASSERTIONS
 
-#include "runtime_stack_check.js"
-
 var __ATPRERUN__  = []; // functions called before the runtime is initialized
 var __ATINIT__    = []; // functions called during startup
 #if HAS_MAIN
@@ -188,12 +178,6 @@ var __ATPOSTRUN__ = []; // functions called after the main() is called
 var __RELOC_FUNCS__ = [];
 #endif
 
-var runtimeInitialized = false;
-
-#if EXIT_RUNTIME
-var runtimeExited = false;
-#endif
-
 function preRun() {
 #if ASSERTIONS && PTHREADS
   assert(!ENVIRONMENT_IS_PTHREAD); // PThreads reuse the runtime from the main thread.
@@ -483,8 +467,6 @@ function abort(what) {
   throw e;
 }
 
-#include "memoryprofiler.js"
-
 #if ASSERTIONS && !('$FS' in addedLibraryItems)
 // show errors on likely calls to FS when it was not included
 var FS = {
@@ -506,8 +488,6 @@ Module['FS_createDataFile'] = FS.createDataFile;
 Module['FS_createPreloadedFile'] = FS.createPreloadedFile;
 #endif
 
-#include "URIUtils.js"
-
 #if ASSERTIONS
 function createExportWrapper(name, nargs) {
   return (...args) => {
@@ -524,8 +504,6 @@ function createExportWrapper(name, nargs) {
 }
 #endif
 
-#include "runtime_exceptions.js"
-
 #if ABORT_ON_WASM_EXCEPTIONS
 // `abortWrapperDepth` counts the recursion level of the wrapper function so
 // that we only handle exceptions at the top level letting the exception
@@ -675,16 +653,6 @@ async function getWasmBinary(binaryFile) {
   return getBinarySync(binaryFile);
 }
 
-#if LOAD_SOURCE_MAP
-var wasmSourceMap;
-#include "source_map_support.js"
-#endif
-
-#if USE_OFFSET_CONVERTER
-var wasmOffsetConverter;
-#include "wasm_offset_converter.js"
-#endif
-
 #if SPLIT_MODULE
 {{{ makeModuleReceiveWithVar('loadSplitModule', undefined, 'instantiateSync',  true) }}}
 var splitModuleProxyHandler = {
@@ -1127,8 +1095,6 @@ var tempDouble;
 var tempI64;
 #endif
 
-#include "runtime_debug.js"
-
 #if RETAIN_COMPILER_SETTINGS
 var compilerSettings = {{{ JSON.stringify(makeRetainedCompilerSettings()) }}} ;
 

src/preamble_minimal.js

@@ -4,18 +4,6 @@
  * SPDX-License-Identifier: MIT
  */
 
-#if SAFE_HEAP
-#include "runtime_safe_heap.js"
-#endif
-
-#if SHARED_MEMORY && ALLOW_MEMORY_GROWTH
-#include "growableHeap.js"
-#endif
-
-#if USE_ASAN
-#include "runtime_asan.js"
-#endif
-
 #if ASSERTIONS
 /** @type {function(*, string=)} */
 function assert(condition, text) {
@@ -58,39 +46,19 @@ var HEAP8, HEAP16, HEAP32, HEAPU8, HEAPU16, HEAPU32, HEAPF32, HEAPF64,
 #endif
   wasmMemory;
 
-#include "runtime_shared.js"
-
-#if PTHREADS
-#include "runtime_pthread.js"
-#endif
-
-#if IMPORTED_MEMORY
-#include "runtime_init_memory.js"
-#endif // IMPORTED_MEMORY
-
-#include "runtime_stack_check.js"
-
-#if LOAD_SOURCE_MAP
-var wasmSourceMap;
-#include "source_map_support.js"
-#endif
-
-#if USE_OFFSET_CONVERTER
-var wasmOffsetConverter;
-#include "wasm_offset_converter.js"
+#if ASSERTIONS || SAFE_HEAP || USE_ASAN
+var runtimeInitialized = false;
 #endif
 
 #if EXIT_RUNTIME
 var __ATEXIT__    = []; // functions called during shutdown
 var runtimeExited = false;
 #endif
 
-#if ASSERTIONS || SAFE_HEAP || USE_ASAN
-var runtimeInitialized = false;
-#endif
+#include "runtime_shared.js"
 
-#include "memoryprofiler.js"
-#include "runtime_exceptions.js"
-#include "runtime_debug.js"
+#if IMPORTED_MEMORY
+#include "runtime_init_memory.js"
+#endif // IMPORTED_MEMORY
 
 // === Body ===

src/runtime_shared.js

@@ -4,6 +4,37 @@
  * SPDX-License-Identifier: MIT
  */
 
+#include "runtime_stack_check.js"
+#include "runtime_exceptions.js"
+#include "runtime_debug.js"
+#include "memoryprofiler.js"
+
+#if SAFE_HEAP
+#include "runtime_safe_heap.js"
+#endif
+
+#if SHARED_MEMORY && ALLOW_MEMORY_GROWTH
+#include "growableHeap.js"
+#endif
+
+#if USE_ASAN
+#include "runtime_asan.js"
+#endif
+
+#if PTHREADS
+#include "runtime_pthread.js"
+#endif
+
+#if LOAD_SOURCE_MAP
+var wasmSourceMap;
+#include "source_map_support.js"
+#endif
+
+#if USE_OFFSET_CONVERTER
+var wasmOffsetConverter;
+#include "wasm_offset_converter.js"
+#endif
+
 {{{
   // Helper function to export a heap symbol on the module object,
   // if requested.

test/other/codesize/test_codesize_cxx_except_wasm.gzsize

@@ -1 +1 @@
-8329
+8328

test/other/codesize/test_codesize_cxx_except_wasm.jssize

@@ -1 +1 @@
-20266
+20267

test/other/codesize/test_codesize_cxx_except_wasm_exnref.gzsize

@@ -1 +1 @@
-8329
+8328

test/other/codesize/test_codesize_cxx_except_wasm_exnref.jssize

@@ -1 +1 @@
-20266
+20267

test/other/codesize/test_codesize_cxx_lto.gzsize

@@ -1 +1 @@
-8363
+8360

test/other/codesize/test_codesize_cxx_mangle.gzsize

@@ -1 +1 @@
-9375
+9374

test/other/codesize/test_codesize_cxx_wasmfs.gzsize

@@ -1 +1 @@
-3600
+3599