From: Tristan Gingold <gingold@adacore.com>
Date: Fri, 26 Aug 2011 13:37:28 +0000 (+0000)
Subject: 2011-08-26  Tristan Gingold  <gingold@adacore.com>
X-Git-Tag: binutils-2_22-branchpoint~194
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0ac5db1998f7b3d147b3fdc1b01def4f00178547;p=external%2Fbinutils.git

2011-08-26  Tristan Gingold  <gingold@adacore.com>

	* config/tc-alpha.c (s_alpha_linkage): Simplify.  Add comments.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 8add94b..cde536e 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,12 @@
+2011-08-26  Tristan Gingold  <gingold@adacore.com>
+
+	* config/tc-alpha.c (s_alpha_linkage): Simplify.  Add comments.
+
+2011-08-26  Tristan Gingold  <gingold@adacore.com>
+
+	* config/tc-alpha.c (add_to_link_pool): Improve comment.
+	(s_alpha_fp_save): Fix indentation.
+
 2011-08-19  Nick Clifton  <nickc@redhat.com>
 
 	* doc/as.texinfo: Include c-xstormy16.texi.
diff --git a/gas/config/tc-alpha.c b/gas/config/tc-alpha.c
index e9dd0fe..931e54c 100644
--- a/gas/config/tc-alpha.c
+++ b/gas/config/tc-alpha.c
@@ -4701,31 +4701,26 @@ s_alpha_linkage (int ignore ATTRIBUTE_UNUSED)
       p = frag_more (LKP_S_K_SIZE);
       memset (p, 0, LKP_S_K_SIZE);
       fixp = fix_new_exp
-	(frag_now, p - frag_now->fr_literal, LKP_S_K_SIZE, &exp, 0,\
+	(frag_now, p - frag_now->fr_literal, LKP_S_K_SIZE, &exp, 0,
 	 BFD_RELOC_ALPHA_LINKAGE);
 
+      if (alpha_insn_label == NULL)
+	alpha_insn_label = symbol_new
+	  (FAKE_LABEL_NAME, now_seg, (valueT) frag_now_fix (), frag_now);
+
+      /* Create a linkage element.  */
       linkage_fixup = (struct alpha_linkage_fixups *)
 	xmalloc (sizeof (struct alpha_linkage_fixups));
-
       linkage_fixup->fixp = fixp;
-      linkage_fixup->next = 0;
-
-      if (alpha_insn_label == 0)
-	alpha_insn_label = symbol_new
-	  (FAKE_LABEL_NAME, now_seg, (valueT) frag_now_fix (), frag_now);
+      linkage_fixup->next = NULL;
       linkage_fixup->label = alpha_insn_label;
 
-      if (alpha_linkage_fixup_root == 0)
-	{
-	  alpha_linkage_fixup_root = alpha_linkage_fixup_tail = linkage_fixup;
-	  alpha_linkage_fixup_tail->next = 0;
-	}
+      /* Append it to the list.  */
+      if (alpha_linkage_fixup_root == NULL)
+        alpha_linkage_fixup_root = linkage_fixup;
       else
-	{
-	  alpha_linkage_fixup_tail->next = linkage_fixup;
-	  alpha_linkage_fixup_tail = linkage_fixup;
-	  alpha_linkage_fixup_tail->next = 0;
-	}
+        alpha_linkage_fixup_tail->next = linkage_fixup;
+      alpha_linkage_fixup_tail = linkage_fixup;
     }
   demand_empty_rest_of_line ();
 }