Only allow RemoveUnusedSymbols enabled per default on Platform.LINUX

happylynx · Oct 25, 2019 · 4691739 · 4691739
1 parent 0622968
commit 4691739
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/...atevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/image/NativeBootImageViaCC.java b/...atevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/image/NativeBootImageViaCC.java
@@ -71,7 +71,11 @@ private static boolean removeUnusedSymbols() {
         if (SubstrateOptions.RemoveUnusedSymbols.hasBeenSet()) {
             return SubstrateOptions.RemoveUnusedSymbols.getValue();
         }
-        return Platform.includedIn(InternalPlatform.PLATFORM_JNI.class);
+        /*
+         * The Darwin linker sometimes segfaults when option -dead_strip is used. Thus, Linux is the
+         * only platform were RemoveUnusedSymbols can be safely enabled per default.
+         */
+        return Platform.includedIn(Platform.LINUX.class);
     }
 
     class BinutilsCCLinkerInvocation extends CCLinkerInvocation {