2009-08-18 Sterling Augustine <sterling@tensilica.com>

	* config/tc-xtensa.c (xtensa_create_xproperty_segments): Use
	sizeof instead of hard-coded value.
	(add_xt_block_frags): Remove unused local variable.
	(frag_flags_to_number): Change return type to flagword.  Remove
	unused local variable.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index d34e601faceabd2bb04b9a16decbeae6fab8f4a2..4a24a4965fbc43b834db207aad036026062261b8 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,4 +1,12 @@
-2009-08-17  Sterling Augustine  <sterling@jaw.hq.tensilica.com>
+2009-08-18  Sterling Augustine  <sterling@tensilica.com>
+
+       * config/tc-xtensa.c (xtensa_create_xproperty_segments): Use
+       sizeof instead of hard-coded value.
+       (add_xt_block_frags): Remove unused local variable.
+       (frag_flags_to_number): Change return type to flagword.  Remove
+       unused local variable.
+
+2009-08-17  Sterling Augustine  <sterling@tensilica.com>
 
        * config/tc-xtensa.c (xtensa_frob_label): move call to 
        do_align_targets to set frag_type only.

diff --git a/gas/config/tc-xtensa.c b/gas/config/tc-xtensa.c
index 9131c7f79dd28ffce80cd0e46dc938b16db2f6b8..caea25c4829367f71aa92867bef1e1c133795932 100644 (file)
--- a/gas/config/tc-xtensa.c
+++ b/gas/config/tc-xtensa.c
@@ -10490,7 +10490,7 @@ static void add_xt_block_frags
 static bfd_boolean xtensa_frag_flags_is_empty (const frag_flags *);
 static void xtensa_frag_flags_init (frag_flags *);
 static void get_frag_property_flags (const fragS *, frag_flags *);
-static bfd_vma frag_flags_to_number (const frag_flags *);
+static flagword frag_flags_to_number (const frag_flags *);
 static void add_xt_prop_frags (segT, xtensa_block_info **, frag_flags_fn);
 
 /* Set up property tables after relaxation.  */
@@ -10716,7 +10716,7 @@ xtensa_create_xproperty_segments (frag_flags_fn flag_fn,
                                      cur_block->size, 4);
                  md_number_to_chars (&frag_data[8 + i * 12],
                                      frag_flags_to_number (&cur_block->flags),
-                                     4);
+                                     sizeof (flagword));
                  cur_block = cur_block->next;
                }
              frag_wane (frag_now);
@@ -10796,7 +10796,6 @@ add_xt_block_frags (segT sec,
                    frag_predicate property_function,
                    frag_predicate end_property_function)
 {
-  bfd_vma seg_offset;
   fragS *fragP;
 
   /* Build it if needed.  */
@@ -10805,8 +10804,6 @@ add_xt_block_frags (segT sec,
   /* We are either at NULL at the beginning or at the end.  */
 
   /* Walk through the frags.  */
-  seg_offset = 0;
-
   if (seg_info (sec)->frchainP)
     {
       for (fragP = seg_info (sec)->frchainP->frch_root;
@@ -10904,10 +10901,10 @@ get_frag_property_flags (const fragS *fragP, frag_flags *prop_flags)
 }
 
 
-static bfd_vma
+static flagword
 frag_flags_to_number (const frag_flags *prop_flags)
 {
-  bfd_vma num = 0;
+  flagword num = 0;
   if (prop_flags->is_literal)
     num |= XTENSA_PROP_LITERAL;
   if (prop_flags->is_insn)
@@ -11053,7 +11050,6 @@ add_xt_prop_frags (segT sec,
                   xtensa_block_info **xt_block,
                   frag_flags_fn property_function)
 {
-  bfd_vma seg_offset;
   fragS *fragP;
 
   /* Build it if needed.  */
@@ -11064,8 +11060,6 @@ add_xt_prop_frags (segT sec,
   /* We are either at NULL at the beginning or at the end.  */
 
   /* Walk through the frags.  */
-  seg_offset = 0;
-
   if (seg_info (sec)->frchainP)
     {
       for (fragP = seg_info (sec)->frchainP->frch_root; fragP;