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

aarch64: add instruction tests: M-S #1422

Merged

Conversation

nkaretnikov
Copy link
Contributor

@nkaretnikov nkaretnikov commented Apr 29, 2019

This change is Reviewable

@nkaretnikov nkaretnikov mentioned this pull request Apr 29, 2019
@nkaretnikov
Copy link
Contributor Author

See #1366 (comment).

Otherwise, 'scripts/travis_test.sh' fails to work.
@nkaretnikov
Copy link
Contributor Author

@ehennenfent Do I need to do anything about this Travis failure? It failed because a method (in tests) is not present in this chunk. The next PR (#1423), which has all changes, including this one, succeeds.

Copy link
Contributor

@ehennenfent ehennenfent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nkaretnikov Don't worry about the failing unit tests for now. I'll likely edit these PR's to merge into a separate dev branch before merging into master, so we should get some final validation that everything passes.

@ehennenfent ehennenfent changed the base branch from master to merge-aarch64 April 30, 2019 16:12
@ehennenfent ehennenfent merged commit 50340c9 into trailofbits:merge-aarch64 Apr 30, 2019
ekilmer added a commit that referenced this pull request May 17, 2019
* master: (28 commits)
  AArch64: fix ldrb size (#1433)
  System Call Audit (#1384)
  ManticoreBase refactor (#1385)
  Add missing checks for ARM boundaries (#1429)
  aarch64: add instruction tests: T-U (#1423)
  aarch64: add instruction tests: M-S (#1422)
  aarch64: add instruction tests: C-L (#1421)
  aarch64: add instruction tests: A-B (#1420)
  aarch64: add everything except instructions (#1418)
  fixup: support ARM64 in '_reg_name'
  Revert "fixup: remove x86-specific code from '_reg_name'"
  review: avoid wildcard imports
  review: rename the file
  fixup: remove x86-specific code from '_reg_name'
  fixup: do not use relative imports
  Generates a more sensible symbolic default for constructor arguments (#1414)
  aarch64: add instructions
  aarch64: add everything except instructions
  Switches the Travis-CI badge from .org to .com (#1416)
  Performance optimization : use set instead of list (#1415)
  ...
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.

2 participants