track warnings in Native Image build and list their count at the very end of the build

Author: wirthi

substratevm/src/com.oracle.svm.core/src/com/oracle/svm/core/SubstrateOptions.java

@@ -1711,4 +1711,8 @@ private static void validateEnableFallbackCompilation(HostedOptionKey<Boolean> o
     public static boolean canEnableFallbackCompilation() {
         return !EnableFallbackCompilation.getValue() && !useEconomyCompilerConfig();
     }
+
+    @Option(help = "Passes the numbers of warnings that occurred in the driver phase to the builder.", type = OptionType.Debug, stability = OptionStability.STABLE) //
+    public static final HostedOptionKey<Integer> DriverWarningsCount = new HostedOptionKey<>(0);
+
 }

substratevm/src/com.oracle.svm.driver/src/com/oracle/svm/driver/NativeImage.java

@@ -1492,6 +1492,8 @@ private int completeImageBuild() {
         // allow native access for all modules on the image builder module path
         var enableNativeAccessModules = getModulesFromPath(imageBuilderModulePath).keySet();
         imageBuilderJavaArgs.add("--enable-native-access=" + String.join(",", enableNativeAccessModules));
+        // pass the number of warnings to the builder process
+        imageBuilderArgs.add(oH(SubstrateOptions.DriverWarningsCount)+LogUtils.getWarningsCount());
 
         boolean useColorfulOutput = configureBuildOutput();
 

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/ProgressReporter.java

@@ -54,6 +54,7 @@
 import java.util.stream.Stream;
 
 import com.oracle.svm.core.RuntimeAssertionsSupport;
+import com.oracle.svm.util.LogUtils;
 import org.graalvm.nativeimage.ImageSingletons;
 import org.graalvm.nativeimage.hosted.Feature;
 import org.graalvm.nativeimage.impl.ImageSingletonsSupport;
@@ -795,6 +796,7 @@ public void printEpilog(Optional<String> optionalImageName, Optional<NativeImage
         l().a(outcomePrefix(buildOutcome)).a(" generating '").bold().a(imageName).reset().a("' ")
                         .a(buildOutcome.successful() ? "in" : "after").a(" ").a(timeStats).a(".").println();
 
+        printWarningMessages();
         printErrorMessage(optionalUnhandledThrowable, parsedHostedOptions);
     }
 
@@ -806,6 +808,17 @@ private static String outcomePrefix(NativeImageGeneratorRunner.BuildOutcome buil
         };
     }
 
+    private void printWarningMessages() {
+        int warningsCount = LogUtils.getWarningsCount() + SubstrateOptions.DriverWarningsCount.getValue();
+        if (warningsCount == 0) {
+            return;
+        }
+
+        l().println();
+        l().yellowBold().a("The build process encountered ").a(warningsCount).a(warningsCount == 1 ? " warning." : " warnings.").reset().println();
+        l().println();
+    }
+
     private void printErrorMessage(Optional<Throwable> optionalUnhandledThrowable, OptionValues parsedHostedOptions) {
         if (optionalUnhandledThrowable.isEmpty()) {
             return;

substratevm/src/com.oracle.svm.util/src/com/oracle/svm/util/LogUtils.java

@@ -29,6 +29,12 @@
 
 // Checkstyle: Allow raw info or warning printing - begin
 public class LogUtils {
+    /**
+     * Number of warnings seen during image build. Note this is limited to the current process, i.e. there is a split
+     * between Driver and Builder.
+     */
+    private static int warningsCount = 0;
+
     /**
      * Print an info message.
      */
@@ -66,6 +72,7 @@ public static void prefixInfo(String prefix, String format, Object... args) {
      */
     public static void warning(String message) {
         System.out.println("Warning: " + message);
+        warningsCount++;
     }
 
     /**
@@ -87,4 +94,8 @@ public static void warning(String format, Object... args) {
     public static void warningDeprecatedEnvironmentVariable(String environmentVariableName) {
         warning("The " + environmentVariableName + " environment variable is deprecated and might be removed in a future release. Please refer to the GraalVM release notes.");
     }
+
+    public static int getWarningsCount() {
+        return warningsCount;
+    }
 }