From 7099f9f917e81faddb74b18c36125d783ea3d403 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Fri, 6 Jul 2001 08:21:35 +0000 Subject: [PATCH] Catch and ignore empty, ineffectual alignment frags when deciding if a branch can be short. --- gas/ChangeLog | 6 ++++++ gas/config/tc-m68k.c | 20 ++++++++++++++++---- 2 files changed, 22 insertions(+), 4 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 860b8a8e391..d5d4c18f0ff 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,9 @@ +2001-07-06 Nick Clifton + + * config/tc-m68k.c (md_estimate_size_before_relax): Catch and + ignore empty, ineffectual alignment frags when deciding if a + branch can be short. + 2001-07-05 Steve Ellcey * config/tc-ia64.c (special_section): Add SPECIAL_SECTION_INIT_ARRAY diff --git a/gas/config/tc-m68k.c b/gas/config/tc-m68k.c index e14c8cb821c..a15d9f516a9 100644 --- a/gas/config/tc-m68k.c +++ b/gas/config/tc-m68k.c @@ -4694,13 +4694,25 @@ md_estimate_size_before_relax (fragP, segment) fragS *l; stop = symbol_get_frag (fragP->fr_symbol); + for (l = fragP->fr_next; l != stop; l = l->fr_next) - if (l->fr_fix + l->fr_var != 0) - break; - if (l == stop) { - fragP->fr_subtype = TAB (TABTYPE (fragP->fr_subtype), SHORT); + /* Catch empty alignment frags whoes fr_offset field + is an alignment requirement of 2 bytes. The check + below will misinterpret this as evidence that real + code exists between the symbol and the instruction + and so will not convert the short jump into a word + jump. */ + if (l->fr_fix == 0 + && l->fr_var == 1 + && (l->fr_type == rs_align || l->fr_type == rs_align_code)) + continue; + + if (l->fr_fix + l->fr_var != 0) + break; } + if (l == stop) + fragP->fr_subtype = TAB (TABTYPE (fragP->fr_subtype), SHORT); } break; default: -- 2.30.2