Content-Length: 310720 | pFad | https://github.com/python/cpython/pull/131778

E2 GH-130415: Remove redundant `sym_matches_type` calls in the JIT optimizer by brandtbucher · Pull Request #131778 · python/cpython · GitHub
Skip to content

GH-130415: Remove redundant sym_matches_type calls in the JIT optimizer#131778

Merged
brandtbucher merged 1 commit intopython:mainfrom
brandtbucher:jit-opt-type-checks
Mar 27, 2025
Merged

GH-130415: Remove redundant sym_matches_type calls in the JIT optimizer#131778
brandtbucher merged 1 commit intopython:mainfrom
brandtbucher:jit-opt-type-checks

Conversation

@brandtbucher
Copy link
Copy Markdown
Member

@brandtbucher brandtbucher commented Mar 26, 2025

These should already be narrowed by a previous guard. If we made a mistake somewhere, the asserts inside the body of the if will catch it.

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.

I think this should be fine. After all the uops only occur after a guard.

@brandtbucher brandtbucher merged commit 972a295 into python:main Mar 27, 2025
69 checks passed
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

interpreter-core (Objects, Python, Grammar, and Parser dirs) skip news topic-JIT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants









ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/python/cpython/pull/131778

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy