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

Remove asm-text "normalization" from cstest #2467

Open
1 of 5 tasks
Rot127 opened this issue Sep 1, 2024 · 0 comments
Open
1 of 5 tasks

Remove asm-text "normalization" from cstest #2467

Rot127 opened this issue Sep 1, 2024 · 0 comments
Labels
bug Testing Test related issue
Milestone

Comments

@Rot127
Copy link
Collaborator

Rot127 commented Sep 1, 2024

Feature

  • New architecture module
  • Support for processor extension
  • Add more instruction details (elaborated below)
  • Binding support for: language
  • Other (elaborated below)

Describe the feature you'd like

The previous cstest "normalized` the asm string. It removes spaces, replaces tabs, replaces hex with decimal and converts negative numbers to their two's complement positive version.

The last point is very problematic, because we no longer detect incorrect assembly output.

I had to move it into the new cstest, because otherwise too many test cases fail. But this should be removed for all auto-sync architectures.

Additional context

Functions I had to move in suite/cstest/src/helper.c

@Rot127 Rot127 added bug Testing Test related issue labels Sep 1, 2024
@Rot127 Rot127 added this to the v6 - Beta milestone Sep 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Testing Test related issue
Projects
None yet
Development

No branches or pull requests

1 participant