[PATCH] ARM Disassembler fix
Wed Jan 6 00:52:00 GMT 2010
Pedro Alves wrote:
> On Tuesday 05 January 2010 22:17:46, Daniel Gutson wrote:
>> Hi Pedro,
>> Pedro Alves wrote:
>>> On Monday 04 January 2010 20:43:29, Daniel Gutson wrote:
>>> The patch breaks text disassembly on arm non-elf targets. Could
>>> this be restrict to elf, please?
>> I now restricted the value initialization of is_data depending on the
>> obj flavour.
>> ChangeLog is the same.
> (I suggest always pasting the changelog with the patch, even
> when it doesn't change between revisions; it makes it easier
> for the person doing the actual reviewing [and maybe commit]
> to have everything in one place.)
>> Please let me know how does this look now.
> The new test is now surely failing on non-elf targets. :-)
> Could you skip it, please? Something like this should do it:
> #skip: *-*-*coff *-*-pe *-*-wince *-*-*aout* *-*-netbsd *-*-riscix*
DONE. (this is the only difference compared against the last patch).
2010-01-05 Daniel Gutson <email@example.com>
* arm-dis.c (print_insn): Fixed search for next
symbol and data dumping condition, and the
initial mapping symbol state.
* gas/arm/dis-data.d: New test case.
* gas/arm/dis-data.s: New file.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 3181 bytes
Desc: not available
More information about the Binutils