diff options
| author | Alan Modra <amodra@bigpond.net.au> | 2013-05-23 03:35:59 +0000 |
|---|---|---|
| committer | Alan Modra <amodra@bigpond.net.au> | 2013-05-23 03:35:59 +0000 |
| commit | 8813b6a21110296b99a8829620c187afacf60c35 (patch) | |
| tree | 0a81917629e294e1455d1d047c79cf0161f4afce /bfd/format.c | |
| parent | c6cdbe745f2781fffc31741da1283eac1682d32a (diff) | |
| download | binutils-redhat-8813b6a21110296b99a8829620c187afacf60c35.tar.gz | |
* format.c (bfd_check_format_matches): Don't match a target in
targ_selvecs if some other target is a better match. If
targets implement match priority, fall back to the first of
the best matches.
Diffstat (limited to 'bfd/format.c')
| -rw-r--r-- | bfd/format.c | 22 |
1 files changed, 21 insertions, 1 deletions
diff --git a/bfd/format.c b/bfd/format.c index e2afa3b2b4..b8f39ca0f0 100644 --- a/bfd/format.c +++ b/bfd/format.c @@ -376,6 +376,9 @@ bfd_check_format_matches (bfd *abfd, bfd_format format, char ***matching) } } + /* We have more than one equally good match. If any of the best + matches is a target in config.bfd targ_defvec or targ_selvecs, + choose it. */ if (match_count > 1) { const bfd_target * const *assoc = bfd_associated_vector; @@ -385,7 +388,8 @@ bfd_check_format_matches (bfd *abfd, bfd_format format, char ***matching) int i = match_count; while (--i >= 0) - if (matching_vector[i] == right_targ) + if (matching_vector[i] == right_targ + && right_targ->match_priority <= best_match) break; if (i >= 0) @@ -396,6 +400,22 @@ bfd_check_format_matches (bfd *abfd, bfd_format format, char ***matching) } } + /* We still have more than one equally good match, and at least some + of the targets support match priority. Choose the first of the + best matches. */ + if (match_count > 1 && best_count != match_count) + { + int i; + + for (i = 0; i < match_count; i++) + { + right_targ = matching_vector[i]; + if (right_targ->match_priority <= best_match) + break; + } + match_count = 1; + } + /* There is way too much undoing of half-known state here. We really shouldn't iterate on live bfd's. Note that saving the whole bfd and restoring it would be even worse; the first thing |