Don't include internal compiler settings with RETAIN_COMPILER_SETTINGS

This is technically a breaking change so I mentioned it in the
ChangeLog.

Fixes: #25663

Author: sbc100

ChangeLog.md

@@ -20,6 +20,9 @@ See docs/process.md for more on how version tagging works.
 
 4.0.19 (in development)
 -----------------------
+- The `RETAIN_COMPILER_SETTINGS` settings and the corresponding
+  `emscripten_get_compiler_setting` API not longer store or report internal
+  compiler settings (this listed in `setttings_internal.js`). (#25667)
 
 4.0.18 - 10/24/25
 -----------------

src/parseTools.mjs

@@ -806,17 +806,8 @@ function addAtPostRun(code) {
 
 function makeRetainedCompilerSettings() {
   const ret = {};
-  for (const [name, value] of Object.entries(global)) {
-    if (name[0] !== '_' && name == name.toUpperCase()) {
-      if (
-        typeof value == 'number' ||
-        typeof value == 'boolean' ||
-        typeof value == 'string' ||
-        Array.isArray(name)
-      ) {
-        ret[name] = value;
-      }
-    }
+  for (const name of PUBLIC_SETTINGS) {
+    ret[name] = globalThis[name];
   }
   return ret;
 }

src/settings_internal.js

@@ -278,3 +278,6 @@ var OUTPUT_FORMAT = '';
 var LOAD_SOURCE_MAP = false;
 
 var ALIASES = [];
+
+// List of public setting names (Used by RETAIN_COMPILER_SETTINGS)
+var PUBLIC_SETTINGS = [];

test/core/emscripten_get_compiler_setting.c

@@ -10,9 +10,11 @@
 #include <emscripten.h>
 
 int main() {
+  // Test boolean, int, and string settings
   printf("INVOKE_RUN: %ld\n", emscripten_get_compiler_setting("INVOKE_RUN"));
-  assert((unsigned)emscripten_get_compiler_setting("OPT_LEVEL") <= 3);
-  assert((unsigned)emscripten_get_compiler_setting("DEBUG_LEVEL") <= 4);
+  assert((unsigned)emscripten_get_compiler_setting("ASSERTIONS") <= 2);
   printf("CLOSURE_WARNINGS: %s\n", (char*)emscripten_get_compiler_setting("CLOSURE_WARNINGS"));
+
+  printf("EMSCRIPTEN_VERSION: %s\n", (char*)emscripten_get_compiler_setting("EMSCRIPTEN_VERSION"));
 }
 

test/core/emscripten_get_compiler_setting.out

@@ -1,2 +1,3 @@
 INVOKE_RUN: 1
 CLOSURE_WARNINGS: quiet
+EMSCRIPTEN_VERSION: invalid compiler setting: EMSCRIPTEN_VERSION

tools/link.py

@@ -1806,6 +1806,8 @@ def get_full_import_name(name):
                                     'emscripten_stack_get_end']
 
   settings.EMSCRIPTEN_VERSION = utils.EMSCRIPTEN_VERSION
+  if settings.RETAIN_COMPILER_SETTINGS:
+    settings.PUBLIC_SETTINGS = [k for k in settings.attrs.keys() if k not in settings.internal_settings]
   settings.SOURCE_MAP_BASE = options.source_map_base or ''
 
   settings.LINK_AS_CXX = (shared.run_via_emxx or settings.DEFAULT_TO_CXX) and not options.nostdlibxx