Skip to content

gh-GH-131798: optimize jit attribute loads on immutable types - #146449

Merged
kumaraditya303 merged 5 commits into
python:mainfrom
kumaraditya303:jit
Mar 26, 2026
Merged

gh-GH-131798: optimize jit attribute loads on immutable types#146449
kumaraditya303 merged 5 commits into
python:mainfrom
kumaraditya303:jit

Conversation

@kumaraditya303

@kumaraditya303kumaraditya303 commented Mar 26, 2026•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

@bedevere-appbedevere-appBot mentioned this pull request Mar 26, 2026
Open
@kumaraditya303kumaraditya303 changed the title gh-GH-131798: do not add type watchers for immutable typesgh-GH-131798: optimize jit attribute loads on immutable typesMar 26, 2026
@kumaraditya303
kumaraditya303 marked this pull request as ready for review March 26, 2026 09:19
@Fidget-Spinner

Fidget-Spinner commented Mar 26, 2026

Copy link
Copy Markdown
Member

It seems list.append is now folded, so you need to change the test test_call_list_append to fix it. That's a great result!

@kumaraditya303

kumaraditya303 commented Mar 26, 2026

Copy link
Copy Markdown
ContributorAuthor

It seems list.append is now folded, so you need to change the test test_call_list_append to fix it. That's a great result!

Thanks, done.

@kumaraditya303
kumaraditya303 enabled auto-merge (squash) March 26, 2026 09:44
@kumaraditya303
kumaraditya303 merged commit fc1f6b1 into python:mainMar 26, 2026
67 checks passed
@kumaraditya303
kumaraditya303 deleted the jit branch March 26, 2026 10:12
@pablogsalpablogsal mentioned this pull request Apr 4, 2026
Merged
clin1234 pushed a commit to clin1234/cpython that referenced this pull request Apr 16, 2026
@kumaraditya303@clin1234
gh-pythonGH-131798: optimize jit attribute loads on immutable types (p…
a1b5f1d
…ython#146449)
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@kumaraditya303@ljfp
gh-pythonGH-131798: optimize jit attribute loads on immutable types (p…
0b24788
…ython#146449)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Fidget-SpinnerFidget-SpinnerFidget-Spinner approved these changes
@markshannonmarkshannonAwaiting requested review from markshannonmarkshannon is a code owner
@tomasr8tomasr8Awaiting requested review from tomasr8tomasr8 is a code owner
@savannahostrowskisavannahostrowskiAwaiting requested review from savannahostrowskisavannahostrowski is a code owner

Assignees

No one assigned

Labels

skip newstopic-JIT

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kumaraditya303@Fidget-Spinner