public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [google] suppress LIPO notes (issue5356041)
@ 2011-11-04 17:41 Rong Xu
  2011-11-04 18:00 ` Xinliang David Li
  0 siblings, 1 reply; 2+ messages in thread
From: Rong Xu @ 2011-11-04 17:41 UTC (permalink / raw)
  To: reply, gcc-patches

Suppress another LIPO notes. Move it under fopt-info.

For google branch only.

Tested with internal branchmark suite.

2011-11-04   Rong Xu  <xur@google.com>

	* gcc/c-family/c-opts.c: move the notes under fopt-info.

Index: gcc/c-family/c-opts.c
===================================================================
--- gcc/c-family/c-opts.c	(revision 180964)
+++ gcc/c-family/c-opts.c	(working copy)
@@ -1104,8 +1104,9 @@
           > (size_t) PARAM_VALUE (PARAM_MAX_LIPO_MEMORY))) {
     i++;
     do {
-      inform (input_location, "Not importing %s: maximum memory "
-	      "consumption reached", in_fnames[i]);
+      if (flag_opt_info >= OPT_INFO_MED)
+        inform (input_location, "Not importing %s: maximum memory "
+                "consumption reached", in_fnames[i]);
       i++;
     } while (i < num_in_fnames);
     return true;

--
This patch is available for review at http://codereview.appspot.com/5356041

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [google] suppress LIPO notes (issue5356041)
  2011-11-04 17:41 [google] suppress LIPO notes (issue5356041) Rong Xu
@ 2011-11-04 18:00 ` Xinliang David Li
  0 siblings, 0 replies; 2+ messages in thread
From: Xinliang David Li @ 2011-11-04 18:00 UTC (permalink / raw)
  To: Rong Xu; +Cc: reply, gcc-patches

ok.

David

On Fri, Nov 4, 2011 at 10:39 AM, Rong Xu <xur@google.com> wrote:
> Suppress another LIPO notes. Move it under fopt-info.
>
> For google branch only.
>
> Tested with internal branchmark suite.
>
> 2011-11-04   Rong Xu  <xur@google.com>
>
>        * gcc/c-family/c-opts.c: move the notes under fopt-info.
>
> Index: gcc/c-family/c-opts.c
> ===================================================================
> --- gcc/c-family/c-opts.c       (revision 180964)
> +++ gcc/c-family/c-opts.c       (working copy)
> @@ -1104,8 +1104,9 @@
>           > (size_t) PARAM_VALUE (PARAM_MAX_LIPO_MEMORY))) {
>     i++;
>     do {
> -      inform (input_location, "Not importing %s: maximum memory "
> -             "consumption reached", in_fnames[i]);
> +      if (flag_opt_info >= OPT_INFO_MED)
> +        inform (input_location, "Not importing %s: maximum memory "
> +                "consumption reached", in_fnames[i]);
>       i++;
>     } while (i < num_in_fnames);
>     return true;
>
> --
> This patch is available for review at http://codereview.appspot.com/5356041
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-11-04 17:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-11-04 17:41 [google] suppress LIPO notes (issue5356041) Rong Xu
2011-11-04 18:00 ` Xinliang David Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).