MatCat.OpenSource/8sa1-binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

obsolete xc16x

Browse Source 
The xc16x md_apply_fix code is just so broken that in my opinion the
target should never have been accepted, and from a quick look at
commit logs for the target it appears that no one has ever contributed
fixes for anything.  This target has just been a 14 year burden on
global binutils and cgen maintainers.  That's not how free software is
supposed to work.

bfd/
	* config.bfd: Obsolete xc16x.
gas/
	* config/tc-xc16x.c (md_apply_fix): Add FIXME.
This commit is contained in:
Alan Modra 2020-07-01 09:15:34 +09:30
parent 054b336d9a
commit 3443489c65
4 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
bfd/ChangeLog
View File

@ -1,3 +1,7 @@
2020-07-01 Alan Modra <amodra@gmail.com>
* config.bfd: Obsolete xc16x.
2020-06-30 H.J. Lu <hongjiu.lu@intel.com> 2020-06-30 H.J. Lu <hongjiu.lu@intel.com>
* archures.c (bfd_mach_i386_nacl): Removed. * archures.c (bfd_mach_i386_nacl): Removed.

1
bfd/config.bfd
View File

@ -54,6 +54,7 @@ case $targ in
exit 1 exit 1
;; ;;
powerpcle-*-pe | powerpcle-*-winnt* | powerpcle-*-cygwin* | \ powerpcle-*-pe | powerpcle-*-winnt* | powerpcle-*-cygwin* | \
xc16x-*-* | \
null) null)
if test "x$enable_obsolete" != xyes; then if test "x$enable_obsolete" != xyes; then
echo "*** Configuration $targ is obsolete." >&2 echo "*** Configuration $targ is obsolete." >&2

4
gas/ChangeLog
View File

@ -1,3 +1,7 @@
2020-07-01 Alan Modra <amodra@gmail.com>
* config/tc-xc16x.c (md_apply_fix): Add FIXME.
2020-07-01 Alan Modra <amodra@gmail.com> 2020-07-01 Alan Modra <amodra@gmail.com>
* testsuite/gas/all/eqv-dot.d: xfail targets that set linkrelax * testsuite/gas/all/eqv-dot.d: xfail targets that set linkrelax

2
gas/config/tc-xc16x.c
View File

@ -323,6 +323,8 @@ tc_gen_reloc (asection *section ATTRIBUTE_UNUSED, fixS *fixp)
void void
md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED) md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
{ {
/* FIXME: This absolute nonsense apparently is to make relative
jumps "work". Of course, it breaks any other type of fixup. */
if (!strstr (seg->name,".debug")) if (!strstr (seg->name,".debug"))
{ {
if (*valP < 128) if (*valP < 128)