Skip to content

gh-148171: Convert variadic argument opcodes to leave their arguments on the stack (CALL_BUILTIN_FAST_WITH_KEYWORDS)#148366

Merged
Fidget-Spinner merged 4 commits intopython:mainfrom
Sacul0457:Optimize-_CALL_BUILTIN_FAST_WITH_KEYWORDS
Apr 11, 2026
Merged

gh-148171: Convert variadic argument opcodes to leave their arguments on the stack (CALL_BUILTIN_FAST_WITH_KEYWORDS)#148366
Fidget-Spinner merged 4 commits intopython:mainfrom
Sacul0457:Optimize-_CALL_BUILTIN_FAST_WITH_KEYWORDS

Conversation

@Sacul0457
Copy link
Copy Markdown
Contributor

@Sacul0457 Sacul0457 commented Apr 11, 2026

Copy link
Copy Markdown
Member

@Fidget-Spinner Fidget-Spinner left a comment

Choose a reason for hiding this comment

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

Just one quip with the test. Please fix and I will merge. Thanks!

self.assertIn("_CALL_BUILTIN_FAST_WITH_KEYWORDS", uops)
self.assertNotIn("_GUARD_CALLABLE_BUILTIN_FAST_WITH_KEYWORDS", uops)
self.assertGreaterEqual(count_ops(ex, "_POP_TOP_NOP"), 5)
self.assertGreaterEqual(count_ops(ex, "_POP_TOP"), 3)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please remove this, we probably will have more in the future when we optimize more ops.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh alr, gimme a sec

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed :)

@Fidget-Spinner Fidget-Spinner merged commit e872c19 into python:main Apr 11, 2026
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants