Skip to content

Fix 32bit build#2796

Open
Rot127 wants to merge 16 commits intocapstone-engine:nextfrom
Rot127:32bit
Open

Fix 32bit build#2796
Rot127 wants to merge 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 6 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
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.

1 participant