Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added "rsil", fix output (removed the extra comma) #19

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

LordCarCar
Copy link

No description provided.

LordCarCar and others added 5 commits September 19, 2020 03:38
From page 455 of Xtensa (ISA) Reference Manual the opcode for MULSH is 0xB2 not 0xB1. B1 is SRA
AS_ASCIIC | AS_ASCIIZ flags do not get zeroed out when this dictionary is initialized and therefore strings are not displayed properly. May just be an issue with my version of IDA. Ignore if strings show up correctly on your version.
From page 455 of Xtensa (ISA) Reference Manual the opcode for MULSH is 0xB2 not 0xB1. B1 is SRA
AS_ASCIIC | AS_ASCIIZ flags do not get zeroed out when this dictionary is initialized and therefore strings are not displayed properly. May just be an issue with my version of IDA. Ignore if strings show up correctly on your version.
Fixes MULSH opcode and strings decompilation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants