search

radareorg / radare2

UNIX-like reverse engineering framework and command-line toolset
https://www.radare.org/
GNU Lesser General Public License v3.0
20.43k stars 2.98k forks source link

wrong v850 disasm #10423

Open radare opened 6 years ago

radare commented 6 years ago 
rasm2 -a v850 -d f5efbfcd
setf , r29

a comma here doesnt seems fine

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. Considering a lot has changed since its creation, we kindly ask you to check again if the issue you reported is still relevant in the current version of radare2. If it is, update this issue with a comment, otherwise it will be automatically closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 3 years ago

This issue has been automatically closed because marked as stale and it has not been updated since then. If the issue is still relevant, feel free to re-open it or open a new one.

trufae commented 3 years ago

According to r2ghidra this is:

$ r2 -a r2ghidra -b 32 -e asm.cpu=v850 -
[0x00000000]> wx f5efbfcd
[0x00000000]> pd 2
            0x00000000      ld.hu -0x3242[r21], r29
            0x00000004      nop
[0x00000000]>

But there are tons of bad instruction disassemblies in the current v850 disassembler, also i dont understand why are you reopening staled issues when i got blamed several times for this even if the issue wasnt fixed. But anyway i will probably not understand it either.

Now i have SDK, samples and documentation for v850 and i try to find time to improve the support, but totally no hurry for that. so ill just assign this issue to me