Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • D dynamorio
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,467
    • Issues 1,467
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 44
    • Merge requests 44
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • DynamoRIO
  • dynamorio
  • Merge requests
  • !3585

i#3574 binutils decoder/encoder tests: Add binutils decode/encode tests to DynamoRIO.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Hendrik Greving requested to merge i3574-dec-enc-tests-from-binutils into master Apr 30, 2019
  • Overview 51
  • Commits 23
  • Pipelines 0
  • Changes 9

This patch adds a test suite based on binutil's gas test suite. It is not equivalent to binutil's suite. For example, DynamoRio does not have an assembler reading mnemonics.

The suite adds a collection of decoder/encoder tests and compares its disassembler output. The expected disassembler strings have been generated by DynamoRIO at the time this suite was created. Hence, it is not golden reference with respect to binutils. There are minor differences between objdump and DynamoRIO's disassembler output that would need to get resolved. This is a possible future improvement to this suite.

Decoding and encoding errors have been debugged and inspected (xref #3577, #3578, #3581 (closed)). Unsupported instructions are either referenced by these issues, or have been removed from the original binutils test suite. For example, unsupported CPL 0 instructions have been removed.

The suite was moved to third_party to honor GNU GPL. Only the .asm files of this suite are subject to GNU GPL.

Large parts of the regression are currently commented out, specifically AVX-512 (xref i#1312), MPX, SHA, VMX (xref i#3581) and others (issues mentioned above).

Issue: #3574

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i3574-dec-enc-tests-from-binutils