Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2161)

Issue 5356041: [google] suppress LIPO notes

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 8 months ago by xur
Modified:
12 years, 8 months ago
Reviewers:
davidxl
CC:
gcc-patches_gcc.gnu.org
Base URL:
svn+ssh://gcc.gnu.org/svn/gcc/branches/google/main/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M gcc/c-family/c-opts.c View 1 chunk +3 lines, -2 lines 0 comments Download

Messages

Total messages: 2
xur
Suppress another LIPO notes. Move it under fopt-info. For google branch only. Tested with internal ...
12 years, 8 months ago (2011-11-04 17:39:49 UTC) #1
davidxl
12 years, 8 months ago (2011-11-04 17:42:34 UTC) #2
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
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b