safely internalize in s_x_x tests too

kripken · kripken · commit 68fa40ad92b6 · 2012-12-21T10:47:40.000-08:00
diff --git a/emcc b/emcc
@@ -1066,7 +1066,7 @@ try:
       # At minimum remove dead functions etc., this potentially saves a lot in the size of the generated code (and the time to compile it)
       exports = ','.join(map(lambda exp: exp[1:], shared.Settings.EXPORTED_FUNCTIONS))
       # internalize carefully, llvm 3.2 will remove even main if not told not to
-      link_opts += ['-internalize', '-internalize-public-api-list=' + exports, '-globaldce']
+      link_opts += shared.Building.get_safe_internalize() + ['-globaldce']
       if DEBUG: print >> sys.stderr, 'emcc: LLVM linktime:', link_opts
       shared.Building.llvm_opt(in_temp(target_basename + '.bc'), link_opts)
       if DEBUG: save_intermediate('dce', 'bc')
diff --git a/tools/shared.py b/tools/shared.py
@@ -921,6 +921,12 @@ def can_use_unsafe_opts():
   def can_inline():
     return Settings.INLINING_LIMIT == 0
 
+  @staticmethod
+  def get_safe_internalize():
+    exports = ','.join(map(lambda exp: exp[1:], Settings.EXPORTED_FUNCTIONS))
+    # internalize carefully, llvm 3.2 will remove even main if not told not to
+    return ['-internalize', '-internalize-public-api-list=' + exports]
+
   @staticmethod
   def pick_llvm_opts(optimization_level):
     '''
@@ -956,7 +962,7 @@ def pick_llvm_opts(optimization_level):
           opts.append('-basicaa') # makes fannkuch slow but primes fast
 
         if Building.can_build_standalone():
-          opts.append('-internalize')
+          opts += Building.get_safe_internalize()
 
         opts.append('-globalopt')
         opts.append('-ipsccp')
