Skip to content

Comments

Fix 32bit build#2796

Merged
Rot127 merged 16 commits intocapstone-engine:nextfrom
Rot127:32bit
Feb 15, 2026
Merged

Fix 32bit build#2796
Rot127 merged 16 commits intocapstone-engine:nextfrom
Rot127:32bit

Conversation

@Rot127
Copy link
Collaborator

@Rot127 Rot127 commented Oct 12, 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

Fixes the 32bit build and enables the tests for it.

  • Remove all usage of va_args and updates Mips, LoongArch, ARM, and PPC to use the ARCH_add_cs_detail_X() functions instead of the generic add_cs_detail().
  • Fixes several bugs related to unspecified type widths and not portable formatting.
  • Adds several cross builds for testing.

Test plan

All Python 32bit builds and tests must be green.

Closing issues

closes #2458
addresses #2416 for v6.

@Rot127
Copy link
Collaborator Author

Rot127 commented Oct 20, 2025

@Antelox Please review as well.

I can't download the test logs (some Github bug as it looks like) to confirm everything went through without bugs.
So mark it as draft until I could.

@Rot127 Rot127 force-pushed the 32bit branch 2 times, most recently from 7ddd723 to 2b3deae Compare February 13, 2026 15:36
@Rot127 Rot127 changed the title 32bit build Fix 32bit build Feb 13, 2026
@Rot127 Rot127 mentioned this pull request Feb 13, 2026
6 tasks
@Rot127 Rot127 marked this pull request as ready for review February 13, 2026 17:12
@Rot127 Rot127 merged commit 3533a9c into capstone-engine:next Feb 15, 2026
35 checks passed
@Rot127 Rot127 deleted the 32bit branch February 15, 2026 16:14
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.

UB on PPC because vargs are not casted.

3 participants