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
  • !5558

i#5326: AArch64 codec: refactor codecsort.py to apply to all codecs

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge i5326-fix-aarch64-codecsort-enums into master Jul 11, 2022
  • Overview 1
  • Commits 2
  • Pipelines 0
  • Changes 2

Created by: joshua-warburton

Previously codecsort.py was only applied to a single codec at a time and thus couldn't allocate enums uniquely as it didn't know about the enum values in other files. This patch simplifies the process of calling codecsort.py so that it can consider all the codecs at once, and thus rewrite them as needed.

It also changes the default mode so that it returns an exit code of 1 if it detects a rewrite is needed making checking the order easier.

Issue: #5326

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i5326-fix-aarch64-codecsort-enums