Skip to content
This repository was archived by the owner on Jun 24, 2026. It is now read-only.

aarch64: add instruction tests: M-S#1422

Merged
ehennenfent merged 10 commits into
trailofbits:merge-aarch64from
nkaretnikov:aarch64-instruction-tests-m-s
Apr 30, 2019
Merged

aarch64: add instruction tests: M-S#1422
ehennenfent merged 10 commits into
trailofbits:merge-aarch64from
nkaretnikov:aarch64-instruction-tests-m-s

Conversation

@nkaretnikov

@nkaretnikov nkaretnikov commented Apr 29, 2019

Copy link
Copy Markdown
Contributor

This change is Reviewable

@nkaretnikov nkaretnikov mentioned this pull request Apr 29, 2019
@nkaretnikov

Copy link
Copy Markdown
Contributor Author

See #1366 (comment).

Otherwise, 'scripts/travis_test.sh' fails to work.
@nkaretnikov

Copy link
Copy Markdown
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.

@ehennenfent ehennenfent left a comment

Copy link
Copy Markdown
Contributor

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants