[patch] make -flto -save-temps less verbose

Cesar Philippidis cesar@codesourcery.com
Thu Mar 13 00:29:00 GMT 2014


I noticed that the lto-wrapper is a little noisy without the -v option
when -save-temps is used. E.g.,

$ gcc main.c -flto -save-temps
[Leaving LTRANS /tmp/ccSEvaB7.args]
[Leaving LTRANS /tmp/ccQomDzb.ltrans.out]
[Leaving LTRANS /tmp/ccVzWdGZ.args]
[Leaving LTRANS /tmp/ccQomDzb.ltrans0.o]

Those messages probably should be suppressed unless the user wants
verbose diagnostics. They also show up as errors in the testsuite
(although none currently use -save-temps with -flto, yet). The attached
patch addresses this issue by disabling those messages unless the user
passes -v to the driver. I've also included a simple test case which
would fail without the change.

Is this OK for stage-4? If so, please check it in since I don't have an
SVN account.

Thanks,
Cesar

-------------- next part --------------
2014-03-12  Cesar Philippidis  <cesar@codesourcery.com>

	gcc/
	* lto-wrapper.c (maybe_unlink_file): Suppress diagnostic
	messages.

	gcc/testsuites/
	* gcc.dg/lto/save-temps_0.c: New file.

Index: gcc/lto-wrapper.c
===================================================================
--- gcc/lto-wrapper.c	(revision 208513)
+++ gcc/lto-wrapper.c	(working copy)
@@ -246,7 +246,7 @@ maybe_unlink_file (const char *file)
 	  && errno != ENOENT)
 	fatal_perror ("deleting LTRANS file %s", file);
     }
-  else
+  else if (verbose)
     fprintf (stderr, "[Leaving LTRANS %s]\n", file);
 }
 
Index: gcc/testsuite/gcc.dg/lto/save-temps_0.c
===================================================================
--- gcc/testsuite/gcc.dg/lto/save-temps_0.c	(revision 0)
+++ gcc/testsuite/gcc.dg/lto/save-temps_0.c	(revision 0)
@@ -0,0 +1,8 @@
+/* { dg-lto-options {{ -O -flto -save-temps}} } */
+/* { dg-lto-do link } */
+
+int
+main (void)
+{
+  return 0;
+}


More information about the Gcc-patches mailing list