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

How to add support to armv8m? #1724

Closed
akiannillo opened this issue Feb 9, 2021 · 8 comments · Fixed by #1949
Closed

How to add support to armv8m? #1724

akiannillo opened this issue Feb 9, 2021 · 8 comments · Fixed by #1949

Comments

@akiannillo
Copy link

I'd like to add armv8m (main line) instruction set and use capstone.
How can I do that?

@XVilka
Copy link
Contributor

XVilka commented Feb 23, 2021

See #1319
Basically, you need to synchronize with the latest LLVM code (TableGen definitions).

See also #1712, #1650, and #1697

@akiannillo
Copy link
Author

I'm not an expert on LLVM.
Can you guide me on where the TableGen definitions are in both projects?

@janek202
Copy link

Hi,

Were you able to move forward with this?
I'm interested in this architecture as well.

@akiannillo
Copy link
Author

No, I was not.
Unfortunately, I have no time to acquire the knowledge to do it by myself.
Any idea from your side?

@janek202
Copy link

No, unfortunately, this is all new to me.
Currently, I don't have enough time to do it by myself.

@akiannillo
Copy link
Author

is there any update on this?

@MostBaffled
Copy link

Capstone v4.0.2 doesn't decode vldr (amongst others) correctly. It just results in ldc instructions (which are related). Is this likely to fixed by syncing to the latest LLVM TableGen definitions as suggested above?

Latest (well, pretty recent) llvm-objdump does correctly disassemble the instructions in question.

@Rot127
Copy link
Collaborator

Rot127 commented May 31, 2023

Will be done with #1949

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 a pull request may close this issue.

5 participants