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.

Sign up for GitHub

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

Jump to bottom

Change Immediate and Memory to support unsized values #228

Closed
postmodern opened this issue Mar 21, 2025 · 0 comments
Closed

Change Immediate and Memory to support unsized values #228

postmodern opened this issue Mar 21, 2025 · 0 comments
Assignees
@postmodern
Labels
enhancement New feature or request internal-api Internal API
Milestone
1.0.0

Comments

@postmodern
Copy link
Member

Instead of trying to infer the size of an Immediate or Memory value, simply leave size as nil. This will allow outputting assembly syntax without the size specifiers.

Since some of the type attributes are calculated from size, we'll have to assign default values to type if size is not known/given.
@postmodern postmodern added enhancement New feature or request internal-api Internal API labels Mar 21, 2025
@postmodern postmodern added this to the 1.0.0 milestone Mar 21, 2025
@postmodern postmodern self-assigned this Mar 21, 2025
@postmodern postmodern moved this from Todo to In Progress in ronin-asm refactor Mar 21, 2025
@postmodern postmodern changed the title Change Immediate and Memory to support nil size values Change Immediate and Memory to support unsized values Mar 21, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in ronin-asm refactor Mar 22, 2025
postmodern added a commit that referenced this issue Mar 22, 2025
@postmodern
Support unsized Immediate and Memory types (closes #228,#227).
0a7ce15
postmodern added a commit that referenced this issue Mar 24, 2025
@postmodern
Support unsized Immediate and Memory types (closes #228,#227).
5937f3d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@postmodern postmodern

Labels
enhancement New feature or request internal-api Internal API
Projects
ronin-asm refactor
Status: Done
Milestone
1.0.0
Development

No branches or pull requests
1 participant
@postmodern