Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-106213: Shorten Emscripten wasm-gc trampoline by a little#133984

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

hoodmane
Copy link
Contributor

@hoodmanehoodmane commentedMay 13, 2025•
edited
Loading

I found out about theif instruction which lets us have slightly shorter code here.

I found out about the `if` instruction which lets us have slightlyshorter code here.
@hoodmanehoodmaneforce-pushed theshorter-emscripten-wasm-gc-trampoline branch from5680660 tode6ffa6CompareMay 13, 2025 20:38
@hoodmanehoodmane changed the titlegh-121700 Shorten Emscripten trampolines by a littlegh-106213 Shorten Emscripten wasm-gc trampolines by a littleMay 13, 2025
@hoodmanehoodmane changed the titlegh-106213 Shorten Emscripten wasm-gc trampolines by a littlegh-106213 Shorten Emscripten wasm-gc trampoline by a littleMay 13, 2025
@freakboy3742freakboy3742 added the needs backport to 3.14bugs and security fixes labelMay 20, 2025
@freakboy3742freakboy3742 changed the titlegh-106213 Shorten Emscripten wasm-gc trampoline by a littlegh-106213: Shorten Emscripten wasm-gc trampoline by a littleMay 20, 2025
Copy link
Contributor

@freakboy3742freakboy3742 left a comment

Choose a reason for hiding this comment

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

👍

@freakboy3742freakboy3742 merged commit3b7888b intopython:mainMay 20, 2025
46 checks passed
@miss-islington-app
Copy link

Thanks@hoodmane for the PR, and@freakboy3742 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestMay 20, 2025
…ythonGH-133984)Using the if instruction results in slightly shorter trampoline code.(cherry picked from commit3b7888b)Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
@bedevere-app
Copy link

GH-134376 is a backport of this pull request to the3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14bugs and security fixes labelMay 20, 2025
freakboy3742 pushed a commit that referenced this pull requestMay 20, 2025
…H-133984) (#134376)Using the if instruction results in slightly shorter trampoline code.(cherry picked from commit3b7888b)Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@freakboy3742freakboy3742freakboy3742 approved these changes

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

Successfully merging this pull request may close these issues.

2 participants
@hoodmane@freakboy3742
[8]ページ先頭
©2009-2025 Movatter.jp