Skip to content

[RISCV] Use range-based for loops in RISCVOptWInstrs. NFC #69647

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
Merged
merged 1 commit into from
Oct 20, 2023
Merged

[RISCV] Use range-based for loops in RISCVOptWInstrs. NFC #69647

merged 1 commit into from
Oct 20, 2023

Conversation

topperc
Copy link
Collaborator

@topperc topperc commented Oct 19, 2023

No description provided.

@topperc topperc requested review from preames and mshockwave October 19, 2023 22:09
@topperc topperc requested review from asb and wangpc-pp October 20, 2023 16:29
asb
asb approved these changes Oct 20, 2023
View reviewed changes
Copy link
Contributor

@asb asb left a comment

Choose a reason for hiding this comment

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

LGTM.

@topperc topperc merged commit e40b2d4 into llvm:main Oct 20, 2023
@spupyrev spupyrev mentioned this pull request Oct 23, 2023
Merged
@banach-space banach-space mentioned this pull request Oct 24, 2023
Merged
@topperc topperc deleted the pr/range-loop branch October 24, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asb asb asb approved these changes

@preames preames Awaiting requested review from preames

@mshockwave mshockwave Awaiting requested review from mshockwave

@wangpc-pp wangpc-pp Awaiting requested review from wangpc-pp

Assignees
No one assigned
Labels
backend:RISC-V
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@topperc @asb @llvmbot