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

Fix CAPSTONE_HAS_PPC flags #1361

Merged

Conversation

knightsc
Copy link
Contributor

@knightsc knightsc commented Feb 2, 2019

The actual conditional PPC flag used is CAPSTONE_HAS_POWERPC. It
appears as if some of the test suite files were using the incorrect flag. Because
of this test_basic, test_detail and test_iter were all failing to output PPC
code in the default configuration.

The actual conditional PPC flag used is CAPSTONE_HAS_POWERPC. It
appears as if some of the test suite files was using the incorrect flag. Because
of this test_basic, test_detail and test_iter were all failing to output PPC
code in the default configuration.
@aquynh aquynh merged commit 65cad8c into capstone-engine:master Feb 3, 2019
@aquynh
Copy link
Collaborator

aquynh commented Feb 3, 2019

merged, thanks!

aquynh pushed a commit that referenced this pull request Feb 3, 2019
The actual conditional PPC flag used is CAPSTONE_HAS_POWERPC. It
appears as if some of the test suite files was using the incorrect flag. Because
of this test_basic, test_detail and test_iter were all failing to output PPC
code in the default configuration.
aquynh pushed a commit that referenced this pull request Feb 3, 2019
The actual conditional PPC flag used is CAPSTONE_HAS_POWERPC. It
appears as if some of the test suite files was using the incorrect flag. Because
of this test_basic, test_detail and test_iter were all failing to output PPC
code in the default configuration.
@knightsc knightsc deleted the fix-basic-and-detail-ppc-tests branch February 4, 2019 17:08
@riptl riptl mentioned this pull request Jul 22, 2022
6 tasks
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.

None yet

2 participants