public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Fix thinko in tree-nested.c
@ 2007-10-26 21:39 Eric Botcazou
  0 siblings, 0 replies; only message in thread
From: Eric Botcazou @ 2007-10-26 21:39 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 401 bytes --]

The head comment of get_trampoline_type reads: "Build or return the type used 
to represent a nested function trampoline" but the function never returns 
without first building a brand new type.

Bootstrapped/regtested on i586-suse-linux, applied on the mainline as obvious.


2007-10-26  Eric Botcazou  <ebotcazou@adacore.com>

	* tree-nested.c (get_trampoline_type): Fix thinko.


-- 
Eric Botcazou

[-- Attachment #2: p.diff --]
[-- Type: text/x-diff, Size: 950 bytes --]

Index: tree-nested.c
===================================================================
--- tree-nested.c	(revision 129646)
+++ tree-nested.c	(working copy)
@@ -402,8 +402,8 @@ static GTY(()) tree trampoline_type;
 static tree
 get_trampoline_type (void)
 {
-  tree record, t;
   unsigned align, size;
+  tree t;
 
   if (trampoline_type)
     return trampoline_type;
@@ -425,12 +425,12 @@ get_trampoline_type (void)
   DECL_ALIGN (t) = align;
   DECL_USER_ALIGN (t) = 1;
 
-  record = make_node (RECORD_TYPE);
-  TYPE_NAME (record) = get_identifier ("__builtin_trampoline");
-  TYPE_FIELDS (record) = t;
-  layout_type (record);
+  trampoline_type = make_node (RECORD_TYPE);
+  TYPE_NAME (trampoline_type) = get_identifier ("__builtin_trampoline");
+  TYPE_FIELDS (trampoline_type) = t;
+  layout_type (trampoline_type);
 
-  return record;
+  return trampoline_type;
 }
 
 /* Given DECL, a nested function, find or create a field in the non-local

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-10-26 21:21 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-10-26 21:39 Fix thinko in tree-nested.c Eric Botcazou

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).