This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix compiler warnings

On Wed, May 06, 2015 at 01:47:38PM +0000, Aditya K wrote:
> I recently compiled gcc with clang and found few useful warnings (,
> I have a patch to fix some of those, it passes bootstrap, please apply these if it is useful.

First some general comments:

You are fixing several unrelated issues, so you should be making one
commit per issue.  That way the people who know an area only need to
review changes to that area.

Its generally prefered you send patches inline instead of as attachments
(you can use git send-email to deal with a lot of administrivia for you)

The ChangeLog isn't correctly formated you can use contrib/mklog and
compare against the existing stuff in gcc/ChangeLog.

diff --git a/gcc/gcov-tool.c b/gcc/gcov-tool.c
index fd27d7c..8261369 100644
--- a/gcc/gcov-tool.c
+++ b/gcc/gcov-tool.c
@@ -193,7 +193,7 @@ static int
 do_merge (int argc, char **argv)
   int opt;
-  int ret;
+  int ret = 0;

This error is actually bogus because merge_usage() calls exit() so the
only actual path to return ret; involves setting ret.  That said I
personally think it would be a cleanup to rewrite the code like this.

while (...)

if (...)
  merge_usage ();

int ret = merge_profiles ();
return ret;

index ce4aca2..2b170e1 100644
--- a/gcc/ipa-icf.c
+++ b/gcc/ipa-icf.c
@@ -505,7 +505,7 @@ sem_item::hash_referenced_symbol_properties (symtab_node *ref,
   if (is_a <cgraph_node *> (ref))
-      if ((!type == FUNC || address || !opt_for_fn (decl, optimize_size))
+      if ((type != FUNC || address || !opt_for_fn (decl, optimize_size))

I'll leave it to honza and marxin if this is what that code meant to do.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]