diff options
| author | Alan Modra <amodra@gmail.com> | 2019-04-15 21:51:44 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2019-04-16 17:12:09 +0930 |
| commit | 871a6bd2d852b0fb677386e1be78c3f4c6939b48 (patch) | |
| tree | f1a378f0c442384f758827ba5c8f5a85ca83d3bd /gas/config/tc-mips.c | |
| parent | 02e902e1a1ec7b74125f329b3faef1992efb6d51 (diff) | |
| download | binutils-gdb-871a6bd2d852b0fb677386e1be78c3f4c6939b48.tar.gz | |
Make frag fr_fix unsigned
The field only stores unsigned values, so let's make it unsigned to
stop people worrying about the possibility of negative values.
* frags.h (struct frag <fr_fix>): Use unsigned type.
* frags.c (frag_new): Assert that current size exceeds
old_frags_var_max_size.
* ehopt.c (get_cie_info): Adjust for unsigned fr_fix.
* listing.c (calc_hex): Likewise.
* write.c (cvt_frag_to_fill, write_relocs): Likewise.
* config/tc-arc.c (md_convert_frag): Likewise.
* config/tc-avr.c (avr_patch_gccisr_frag): Likewise.
* config/tc-mips.c (md_convert_frag): Likewise.
* config/tc-rl78.c (md_convert_frag): Likewise.
* config/tc-rx.c (md_convert_frag): Likewise.
* config/tc-sparc.c (md_apply_fix): Likewise.
* config/tc-xtensa.c (next_instrs_are_b_retw): Likewise.
(unrelaxed_frag_min_insn_count, unrelaxed_frag_has_b_j): Likewise.
Diffstat (limited to 'gas/config/tc-mips.c')
| -rw-r--r-- | gas/config/tc-mips.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gas/config/tc-mips.c b/gas/config/tc-mips.c index 7eab392ace2..70c6548283e 100644 --- a/gas/config/tc-mips.c +++ b/gas/config/tc-mips.c @@ -19056,7 +19056,7 @@ md_convert_frag (bfd *abfd ATTRIBUTE_UNUSED, segT asec, fragS *fragp) relax_substateT subtype = fragp->fr_subtype; bfd_boolean second_longer = (subtype & RELAX_SECOND_LONGER) != 0; bfd_boolean use_second = (subtype & RELAX_USE_SECOND) != 0; - int first, second; + unsigned int first, second; fixS *fixp; first = RELAX_FIRST (subtype); @@ -19099,7 +19099,7 @@ md_convert_frag (bfd *abfd ATTRIBUTE_UNUSED, segT asec, fragS *fragp) sequence instead. */ while (fixp && fixp->fx_frag == fragp - && fixp->fx_where < fragp->fr_fix - second) + && (valueT) fixp->fx_where + second < fragp->fr_fix) { if (subtype & RELAX_USE_SECOND) fixp->fx_done = 1; |