lto-cgraph.c (input_node_opt_summary): Remove unused code.

gcc/
2012-10-08  Tobias Burnus  <burnus@net-b.de>

	* lto-cgraph.c (input_node_opt_summary): Remove unused code.
	* lto-opts.c (append_to_collect_gcc_options): Fix condition.
	* lto-symtab.c (lto_cgraph_replace_node): Don't xstrdup string
	which is passed to fprintf.

gcc/lto/
2012-10-08  Tobias Burnus  <burnus@net-b.de>

	* lto-lang.c (lto_register_builtin_type): Avoid useless
	decl creation.
	* lto-object.c (lto_obj_file_open, lto_obj_file_open): Free memory.

From-SVN: r192250

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9967ecf3f60940d365b2e83399bfc5412d039266..7ad8a41c19735a9400d7d7e88d26983ca20b15c7 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,10 @@
+2012-10-09  Tobias Burnus  <burnus@net-b.de>
+
+       * lto-cgraph.c (input_node_opt_summary): Remove unused code.
+       * lto-opts.c (append_to_collect_gcc_options): Fix condition.
+       * lto-symtab.c (lto_cgraph_replace_node): Don't xstrdup string
+       which is passed to fprintf.
+
 2012-10-09  Paolo Carlini  <paolo.carlini@oracle.com>
 
        PR c++/54194

diff --git a/gcc/lto-cgraph.c b/gcc/lto-cgraph.c
index 4f952f59bf9f8230a34b7bb968a71b722e9dcf89..b23f192e3bd6042e346c3c74a58cf05359b63790 100644 (file)
--- a/gcc/lto-cgraph.c
+++ b/gcc/lto-cgraph.c
@@ -1517,14 +1517,9 @@ input_node_opt_summary (struct cgraph_node *node,
   count = streamer_read_uhwi (ib_main);
   for (i = 0; i < count; i++)
     {
-      int parm_num;
-      tree parm;
       struct ipa_replace_map *map = ggc_alloc_ipa_replace_map ();
 
       VEC_safe_push (ipa_replace_map_p, gc, node->clone.tree_map, map);
-      for (parm_num = 0, parm = DECL_ARGUMENTS (node->symbol.decl); parm_num;
-          parm = DECL_CHAIN (parm))
-       parm_num --;
       map->parm_num = streamer_read_uhwi (ib_main);
       map->old_tree = NULL;
       map->new_tree = stream_read_tree (ib_main, data_in);

diff --git a/gcc/lto-opts.c b/gcc/lto-opts.c
index 668a5ce9163d45bca900d739d145504ba30f33a1..a235f41da8de399511b169525e9c9333a01e017e 100644 (file)
--- a/gcc/lto-opts.c
+++ b/gcc/lto-opts.c
@@ -44,7 +44,7 @@ append_to_collect_gcc_options (struct obstack *ob,
                               bool *first_p, const char *opt)
 {
   const char *p, *q = opt;
-  if (!first_p)
+  if (!*first_p)
     obstack_grow (ob, " ", 1);
   obstack_grow (ob, "'", 1);
   while ((p = strchr (q, '\'')))

diff --git a/gcc/lto-symtab.c b/gcc/lto-symtab.c
index 25c0b22dc485784ef3879ba9c5ca1260f4b39545..659a68b68a52dc7dc5a0d04400dec435a0092b93 100644 (file)
--- a/gcc/lto-symtab.c
+++ b/gcc/lto-symtab.c
@@ -46,8 +46,8 @@ lto_cgraph_replace_node (struct cgraph_node *node,
     {
       fprintf (cgraph_dump_file, "Replacing cgraph node %s/%i by %s/%i"
               " for symbol %s\n",
-              xstrdup (cgraph_node_name (node)), node->uid,
-              xstrdup (cgraph_node_name (prevailing_node)),
+              cgraph_node_name (node), node->uid,
+              cgraph_node_name (prevailing_node),
               prevailing_node->uid,
               IDENTIFIER_POINTER ((*targetm.asm_out.mangle_assembler_name)
                 (IDENTIFIER_POINTER (DECL_ASSEMBLER_NAME (node->symbol.decl)))));

diff --git a/gcc/lto/ChangeLog b/gcc/lto/ChangeLog
index d9849b476988dfac7a415117d7f7a06276e49d8e..265922f5d133e2f3287d8b866b85997d7cb5c806 100644 (file)
--- a/gcc/lto/ChangeLog
+++ b/gcc/lto/ChangeLog
@@ -1,3 +1,9 @@
+2012-10-09  Tobias Burnus  <burnus@net-b.de>
+
+       * lto-lang.c (lto_register_builtin_type): Avoid useless
+       decl creation.
+       * lto-object.c (lto_obj_file_open, lto_obj_file_open): Free memory.
+
 2012-10-08  Tobias Burnus  <burnus@net-b.de>
 
        * lto.c (lto_wpa_write_files, read_cgraph_and_symbols):

diff --git a/gcc/lto/lto-lang.c b/gcc/lto/lto-lang.c
index 280d88359deae2a727a9973273a8a0a4cbb91197..edfab74de0ad5b16acc71a3fe2ed71ce6dff9909 100644 (file)
--- a/gcc/lto/lto-lang.c
+++ b/gcc/lto/lto-lang.c
@@ -1079,10 +1079,13 @@ lto_register_builtin_type (tree type, const char *name)
 {
   tree decl;
 
-  decl = build_decl (UNKNOWN_LOCATION, TYPE_DECL, get_identifier (name), type);
-  DECL_ARTIFICIAL (decl) = 1;
   if (!TYPE_NAME (type))
-    TYPE_NAME (type) = decl;
+    {
+      decl = build_decl (UNKNOWN_LOCATION, TYPE_DECL,
+                        get_identifier (name), type);
+      DECL_ARTIFICIAL (decl) = 1;
+      TYPE_NAME (type) = decl;
+    }
 
   registered_builtin_types = tree_cons (0, type, registered_builtin_types);
 }

diff --git a/gcc/lto/lto-object.c b/gcc/lto/lto-object.c
index daf3bd002a32a6781b2e9a3a219bf71d84d0cd58..ce24400cb108f21f619965afbba968e5bc777d51 100644 (file)
--- a/gcc/lto/lto-object.c
+++ b/gcc/lto/lto-object.c
@@ -133,7 +133,10 @@ lto_obj_file_open (const char *filename, bool writable)
          errmsg = simple_object_attributes_merge (saved_attributes, attrs,
                                                   &err);
          if (errmsg != NULL)
-           goto fail_errmsg;
+           {
+             free (attrs);
+             goto fail_errmsg;
+           }
        }
     }
   else
@@ -155,11 +158,13 @@ lto_obj_file_open (const char *filename, bool writable)
     error ("%s: %s: %s", fname, errmsg, xstrerror (err));
                                         
  fail:
-  if (lo != NULL)
+  if (lo->fd != -1)
     lto_obj_file_close ((lto_file *) lo);
+  free (lo);
   return NULL;
 }
 
+
 /* Close FILE.  If FILE was opened for writing, it is written out
    now.  */