Skip to content

Conversation

@Rot127
Copy link
Collaborator

@Rot127 Rot127 commented Jul 11, 2025

Your checklist for this pull request

  • I've documented or updated the documentation of every API function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

Corrections of the docs @moste00 found during refactoring RISCV

  • Describe ArchLinkage.h in more detail.
  • Add hint about MCUpdate that the Assembler can be used as well if not tests generate.

Test plan

...

Closing issues

...

@moste00
Copy link
Contributor

moste00 commented Jul 12, 2025

@Rot127 should there an implied relationship between the vendor/llvm_root and the capstone root, or are they indepedent directories on the same level?

image

@Rot127
Copy link
Collaborator Author

Rot127 commented Jul 12, 2025

Good catch.

The dir to clone LLVM into is:
suite/auto-sync/vendor/

llvm_root is the directory. Of course you can clone LLVM somewhere else and just add an llvm_root link

This should be made clearer.

@Rot127 Rot127 marked this pull request as ready for review November 9, 2025 11:53
@Rot127 Rot127 merged commit 24a2248 into capstone-engine:next Nov 10, 2025
23 checks passed
@Rot127 Rot127 deleted the doc-fixups branch November 10, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants