wasm-backend: Don't export symbols by default from MAIN/SIDE_MODULEs

This matches the current behavior of fastcomp since #7371 disabled
EXPORT_ALL for MAIN/SIDE_MODULEs.

Since most of our dylink tests set EXPORT_ALL this didn't get noticed.

Author: sbc100

tools/shared.py

@@ -1920,28 +1920,29 @@ def link_lld(args, target, opts=[], lto_level=0):
                                      not Settings.ASYNCIFY):
       cmd.append('--strip-debug')
 
-    if Settings.RELOCATABLE:
-      if Settings.MAIN_MODULE == 2 or Settings.SIDE_MODULE == 2:
-        cmd.append('--no-export-dynamic')
-      else:
-        cmd.append('--no-gc-sections')
-        cmd.append('--export-dynamic')
-
-    if Settings.LINKABLE:
+    #if Settings.RELOCATABLE:
+      #if Settings.MAIN_MODULE == 2 or Settings.SIDE_MODULE == 2:
+      #cmd.append('--no-export-dynamic')
+      #else:
+        #cmd.append('--no-gc-sections')
+        #cmd.append('--export-dynamic')
+
+    if Settings.EXPORT_ALL:
       cmd.append('--export-all')
+    else:
+      # in standalone mode, crt1 will call the constructors from inside the wasm
+      if not Settings.STANDALONE_WASM:
+        cmd += ['--export', '__wasm_call_ctors']
 
-    # in standalone mode, crt1 will call the constructors from inside the wasm
-    if not Settings.STANDALONE_WASM:
-      cmd += ['--export', '__wasm_call_ctors']
-
-    cmd += ['--export', '__data_end']
+      cmd += ['--export', '__data_end']
 
-    for export in Settings.EXPORTED_FUNCTIONS:
-      cmd += ['--export', export[1:]] # Strip the leading underscore
+      for export in Settings.EXPORTED_FUNCTIONS:
+        cmd += ['--export', export[1:]] # Strip the leading underscore
 
     if Settings.RELOCATABLE:
       if Settings.SIDE_MODULE:
         cmd.append('-shared')
+        cmd.append('--no-export-dynamic')
       else:
         cmd.append('-pie')