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

YJIT: Rest and block_arg support #7584

Merged

Conversation

jimmyhmiller
Copy link
Contributor

This reverts commit 30e7561.

#7557 was originally reverted because it was believed to be the cause of the sequel benchmark failing. The underlying issue was actually fixed in #7575

@matzbot matzbot requested a review from a team March 23, 2023 16:47
@nobu
Copy link
Member

nobu commented Mar 23, 2023

The message "Revert "Revert ... feels very silly to me.

@jimmyhmiller jimmyhmiller force-pushed the yjit-restore-rest-and-block-support branch from e9fa3db to 4487012 Compare March 23, 2023 16:57
@jimmyhmiller jimmyhmiller changed the title Revert "Revert "YJIT: Rest and block_arg support (#7557)"" YJIT: Rest and block_arg support Mar 23, 2023
@jimmyhmiller
Copy link
Contributor Author

@nobu Sorry that was just the default title from running git revert. Just changed it back to the original title of the original PR.

@jimmyhmiller jimmyhmiller force-pushed the yjit-restore-rest-and-block-support branch from 4487012 to c4ff415 Compare March 24, 2023 18:34
@jimmyhmiller jimmyhmiller force-pushed the yjit-restore-rest-and-block-support branch from c4ff415 to 1520091 Compare March 24, 2023 20:07
@maximecb maximecb merged commit 59c3fac into ruby:master Mar 24, 2023
97 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants