Skip to content

llvm-reduce: Handle cloning for MachineJumpTableInfo #69086

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 31, 2023
Merged

llvm-reduce: Handle cloning for MachineJumpTableInfo #69086

merged 1 commit into from
Oct 31, 2023

Conversation

arsenm
Copy link
Contributor

@arsenm arsenm commented Oct 15, 2023

No description provided.

@arsenm arsenm requested a review from aemerson October 15, 2023 01:51
@github-actions
Copy link

github-actions bot commented Oct 15, 2023 •
edited
Loading

✅ With the latest revision this PR passed the C/C++ code formatter.

@arsenm arsenm force-pushed the llvm-reduce-machinejumptableinfo branch from 1134997 to d986745 Compare October 15, 2023 02:26
aemerson
aemerson approved these changes Oct 31, 2023
View reviewed changes
Copy link
Contributor

@aemerson aemerson left a comment

Choose a reason for hiding this comment

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

Looks like this jump table code is prehistoric, we should convert to using SmallVector in another change.

@arsenm arsenm merged commit 748f861 into llvm:main Oct 31, 2023
@arsenm arsenm deleted the llvm-reduce-machinejumptableinfo branch October 31, 2023 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aemerson aemerson aemerson approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arsenm @aemerson