[PATCH v3 2/2] LD/PE: Remove remains of MIPS target support
Jan Beulich
jbeulich@suse.com
Tue Oct 14 07:18:09 GMT 2025
On 13.10.2025 23:06, Maciej W. Rozycki wrote:
> On Thu, 9 Oct 2025, Jan Beulich wrote:
>
>>> Remove the relevant pieces then, originally from commit 344a211f9995
>>> ("Add support for WinCE based toolchains."), including MIPS architecture
>>> parts and HIGHADJ relocation support in particular. Retain code for LOW
>>> relocation however, even though included with said commit, as it remains
>>> usable by ARM, i386 and x86-64 targets.
>>
>> Okay.
>
> I have committed this change now, thank you for your review.
>
> NB for `mcore-pe' there's still:
>
> XFAIL: PE-COFF 32-bit image fixup
>
> so clearly commit f72471128d1a ("ld/PE: adjust pe_detail_list[]") wasn't
> enough to get this target to work.
This likely falls into the same category as what I hinted at with my remark
"As I lack sufficient knowledge of the target, introduction of
jmp_mcore_bytes[] will need to be handled by someone else" in the description
of the respective patch. I haven't looked at what specifically is the issue
with this (new) test, though - maybe it's easy enough to take care of. For
the tests I added (in the final patch of the series) adding what I added was
sufficient for not having to exclude the target from testing.
Jan
More information about the Binutils
mailing list