[Bug gas/25331] mmix-gas fails to assemble newlib-3.1.0: internal error: fixup not contained within frag

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=25331

--- Comment #4 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot 
gnu.org> ---
The master branch has been updated by Hans-Peter Nilsson <hp@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=b20e7614da3d64ffe05617d5ac89819bff6bedf8

commit b20e7614da3d64ffe05617d5ac89819bff6bedf8
Author: Hans-Peter Nilsson <hp@bitrange.com>
Date:   Mon Jun 29 05:32:02 2020 +0200

    gas: Fix mmix fixups and TC_FX_SIZE_SLACK, PR25331

    Finally; sorry for the delay.  There were a few false starts, where I
    misinterpreted the error-messages and the comment that Alan added:
    it's not the fix size that's too large (and the frag too small), it's
    stating the wrong size of what will be "fixed up" - that of the actual
    target value, not the size of the field that needs to be adjusted.
    Comments added for clarity.

    Test-suite committed separately.

    gas:
            PR gas/25331
            * config/tc-mmix.c (md_assemble) <fixup for
            BFD_RELOC_MMIX_BASE_PLUS_OFFSET>: This fixup affects 1 byte, not 8.
            Also, set its fx_no_overflow.
            (md_convert_frag) <case ENCODE_RELAX (STATE_PUSHJSTUB,
STATE_ZERO)>:
            Similarly this fixup affects 4 bytes, not 8 and needs its
            fx_no_overflow set.
            * config/tc-mmix.h (TC_FX_SIZE_SLACK): Don't define.

-- 
You are receiving this mail because:
You are on the CC list for the bug.