[PATCH v1] Microblaze: Widen mask used in opcodes/microblaze-dis.c

Michael Eager eager@eagercon.com
Sat Jan 4 22:47:43 GMT 2025


On 12/31/24 9:26 PM, Gopi Kumar Bulusu wrote:
> namaskaram,
> 
> Instead of using 0xFFFF0000,
> or with (~0xFFFF) to sign extend negative 16-bit value
> and with (~0xFFFF) to extract higher order address bits
> 
> While testing gdbserver support this was found to cause issues with
> software single stepping due to incorrect target address computation in gdb.
> 
> This is a preparatory patch ahead of gdbserver support for Microblaze
> 
> Files changed:
> opcodes/ChangeLog
> opcodes/microblaze-dis.c
> 
> Build/Test:
> Built for baremetal (microblazeel-elf) target
> Tested with gas/ld/binutils tests (make check)
> No regressions found
> 
> dhanyavadam
> gopi
> 

Hi Gopi --

Do you have write access to the Binutils repo?

If you do, this patch is OK to check in.  Please add a comment
in the git log.

If you don't, I'll apply this when I have an opportunity.


-- 
Michael Eager



More information about the Binutils mailing list