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

feat: Replace LD_SIGN_EXTENDED_32_CONSTANT with CUSTOM_LOAD_UIMM #288

Merged
merged 3 commits into from
Sep 8, 2022

Conversation

xxuejie
Copy link
Collaborator

@xxuejie xxuejie commented Sep 7, 2022

LD_SIGN_EXTENDED_32_CONSTANT is exactly the same as CUSTOM_LOAD_IMM, this change removes LD_SIGN_EXTENDED_32_CONSTANT, then adds CUSTOM_LOAD_UIMM, which as shown in the test case , can be helpful to eliminate all AUIPCs.

LD_SIGN_EXTENDED_32_CONSTANT is exactly the same as CUSTOM_LOAD_IMM,
this change removes LD_SIGN_EXTENDED_32_CONSTANT, then adds
CUSTOM_LOAD_UIMM, which as shown in the test case , can be helpful to
eliminate all AUIPCs.
Copy link
Collaborator

@mohanson mohanson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mohanson mohanson merged commit 4575b0b into develop Sep 8, 2022
@mohanson mohanson deleted the add-uimm-opcode branch September 8, 2022 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants