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

[EVM] Refactor EVMMCInstLower::Lower #745

Open
vladimirradosavljevic opened this issue Dec 6, 2024 · 0 comments
Open

[EVM] Refactor EVMMCInstLower::Lower #745

vladimirradosavljevic opened this issue Dec 6, 2024 · 0 comments
Labels
EVM backend refactoring

Comments

@vladimirradosavljevic
Copy link
Contributor

vladimirradosavljevic commented Dec 6, 2024 •
edited
Loading

Refactor lowerOperand from EVMMCInstLower::Lower similar to this.
Since we need to make an instance of EVMMCInstLower in EVMAsmPrinter and we are using MachineRegisterInfo in EVMMCInstLower, to reduce complexity of passing instance to EVMMCInstLower maybe the easiest solution is to merge EVMMCInstLower into EVMAsmPrinter like here to avoid that.
When this is done, instead of creating MCOperand manually, lowerOperand should be used for that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
EVM backend refactoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vladimirradosavljevic