| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6ae3ba9d-30d4-03c9-a087-cd4882e4986c |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E516:37ACD:2F343D:3F36B9:6969D351 |
| html-safe-nonce | 6317c15dc8a6d9c171e364e67e66f770d49e0d6ce7a78a2d0385d1f7a4271579 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFNTE2OjM3QUNEOjJGMzQzRDozRjM2Qjk6Njk2OUQzNTEiLCJ2aXNpdG9yX2lkIjoiODU4MDAwNzc3MzI5NjI1MTcyOSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | eb71758a7f3c87bb05a2fefe7e3e49f44a871ad57aabcb9a5131c44e865c9216 |
| hovercard-subject-tag | pull_request:1256966373 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///pull_requests/show/files |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/102320/files |
| twitter:image | https://avatars.githubusercontent.com/u/1055913?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1055913?s=400&v=4 |
| og:image:alt | After we merge this I will make a PR to move codegen to a separate file.
Skipping news - we will have one entry for the entire refactor once it's done.
Issue: gh-87092 |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | acedec8b5f975d9e3d494ddd8f949b0b8a0de59d393901e26f73df9dcba80056 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 83c08c21cdda978090dc44364b71aa5bc6dcea79 |
| ui-target | canary-2 |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://github.com/python/cpython/pull/102320/files#start-of-content |
|
| https://github.com/ |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F102320%2Ffiles |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub SparkBuild and deploy intelligent apps | https://github.com/features/spark |
| GitHub ModelsManage and compare prompts | https://github.com/features/models |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| GitHub SponsorsFund open source developers | https://github.com/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/accelerator |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F102320%2Ffiles |
|
Sign up
| https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Ffiles&source=header-repo&source_repo=python%2Fcpython |
| Reload | https://github.com/python/cpython/pull/102320/files |
| Reload | https://github.com/python/cpython/pull/102320/files |
| Reload | https://github.com/python/cpython/pull/102320/files |
|
python
| https://github.com/python |
| cpython | https://github.com/python/cpython |
| Please reload this page | https://github.com/python/cpython/pull/102320/files |
|
Notifications
| https://github.com/login?return_to=%2Fpython%2Fcpython |
|
Fork
33.9k
| https://github.com/login?return_to=%2Fpython%2Fcpython |
|
Star
71.1k
| https://github.com/login?return_to=%2Fpython%2Fcpython |
|
Code
| https://github.com/python/cpython |
|
Issues
5k+
| https://github.com/python/cpython/issues |
|
Pull requests
2.1k
| https://github.com/python/cpython/pulls |
|
Actions
| https://github.com/python/cpython/actions |
|
Projects
31
| https://github.com/python/cpython/projects |
|
Security
Uh oh!
There was an error while loading. Please reload this page.
| https://github.com/python/cpython/security |
| Please reload this page | https://github.com/python/cpython/pull/102320/files |
|
Insights
| https://github.com/python/cpython/pulse |
|
Code
| https://github.com/python/cpython |
|
Issues
| https://github.com/python/cpython/issues |
|
Pull requests
| https://github.com/python/cpython/pulls |
|
Actions
| https://github.com/python/cpython/actions |
|
Projects
| https://github.com/python/cpython/projects |
|
Security
| https://github.com/python/cpython/security |
|
Insights
| https://github.com/python/cpython/pulse |
| Sign up for GitHub
| https://github.com/signup?return_to=%2Fpython%2Fcpython%2Fissues%2Fnew%2Fchoose |
| terms of service | https://docs.github.com/terms |
| privacy statement | https://docs.github.com/privacy |
| Sign in | https://github.com/login?return_to=%2Fpython%2Fcpython%2Fissues%2Fnew%2Fchoose |
| iritkatriel | https://github.com/iritkatriel |
| python:main | https://github.com/python/cpython/tree/main |
| iritkatriel:instruction-stream | https://github.com/iritkatriel/cpython/tree/instruction-stream |
|
Conversation
15
| https://github.com/python/cpython/pull/102320 |
|
Commits
26
| https://github.com/python/cpython/pull/102320/commits |
|
Checks
0
| https://github.com/python/cpython/pull/102320/checks |
|
Files changed
| https://github.com/python/cpython/pull/102320/files |
| Please reload this page | https://github.com/python/cpython/pull/102320/files |
|
gh-87092: compiler's CFG construction moved to after codegen stage
| https://github.com/python/cpython/pull/102320/files#top |
|
Show all changes
26 commits
| https://github.com/python/cpython/pull/102320/files |
|
43df9b3
compiler uses instruction stream to create codeobjects
iritkatriel Feb 25, 2023
| https://github.com/python/cpython/pull/102320/commits/43df9b3a07cf00f691051c33036e482bac14027f |
|
2b43f88
remove unused code
iritkatriel Feb 26, 2023
| https://github.com/python/cpython/pull/102320/commits/2b43f88579514427f7f4a49d7996fa3a5d22bacc |
|
52a1d1d
use newg all the way
iritkatriel Feb 26, 2023
| https://github.com/python/cpython/pull/102320/commits/52a1d1d089a5c95e964539c582a6df07dae6dddc |
|
9b737f1
remove double processing
iritkatriel Feb 26, 2023
| https://github.com/python/cpython/pull/102320/commits/9b737f1d6e8abbeab6ce5a7db535908cc1d9b96a |
|
083b8b1
split cfg_builder_use_label/cfg_builder_addop to separate versions fo…
iritkatriel Feb 26, 2023
| https://github.com/python/cpython/pull/102320/commits/083b8b17371b7e4246913fb1e93044e17537cfcc |
|
a56bafe
codegen functions take an instr_stream
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/a56bafed32cf18778bb03f6ec803a5499c7af08e |
|
8be1b28
free isntr_stream. Attach it to compiluer_unit (not cfg_builder)
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/8be1b288861acd639514b89f1a2f138ae32240e0 |
|
741f773
move things around
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/741f77367f0ea1c68fb315ba5754972a27fe1a82 |
|
fb07525
labels generated by instr_stream instead of cfg_builder
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/fb0752503c25bcd7d2294d034f2adfc16448198c |
|
1b06bf8
add implicit RETURN NONE to the stream
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/1b06bf873325e476fdb2616a02726119716837d5 |
|
142a7f1
remove u_cfg_builder
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/142a7f14129bb40195bf65f1a1738b8b4f61724d |
|
f9f8443
init the cfg_builder in instr_stream_to_cfg
iritkatriel Feb 27, 2023
| https://github.com/python/cpython/pull/102320/commits/f9f844342b87fc00f5a007964dd18950921401e0 |
|
f6fcde9
stream->sequence
iritkatriel Feb 28, 2023
| https://github.com/python/cpython/pull/102320/commits/f6fcde97e448ddd0ce1b6723cd83f5a230691913 |
|
84f5a9d
instr->cfg_instr, codegen_instr->instr
iritkatriel Feb 28, 2023
| https://github.com/python/cpython/pull/102320/commits/84f5a9d0a0b6409bea0bda698ec7e6ce52096690 |
|
cd0225d
make sure we always emit something to jump to at the end
iritkatriel Feb 28, 2023
| https://github.com/python/cpython/pull/102320/commits/cd0225d3aefd9023d626dc3c1d9c76a48a2165c2 |
|
b8f4acf
INSTR_STREAM --> INSTR_SEQUENCE
iritkatriel Feb 28, 2023
| https://github.com/python/cpython/pull/102320/commits/b8f4acf3cc01f02d37a895f92e047572d1b742fd |
|
258f142
free the cfg_builder in the test harness
iritkatriel Mar 1, 2023
| https://github.com/python/cpython/pull/102320/commits/258f14203c141f214c4c5bd3ff54c7529139e3ce |
|
e5653ee
Merge branch 'main' into instruction-stream
iritkatriel Mar 1, 2023
| https://github.com/python/cpython/pull/102320/commits/e5653eecdf5908a88db7dd9fd802230793519a3e |
|
788da74
Merge branch 'main' into instruction-stream
iritkatriel Mar 2, 2023
| https://github.com/python/cpython/pull/102320/commits/788da74bf72200b0528a388c4c02af1dc2812cf6 |
|
fd7f2b3
#ifndef NDEBUG around cfg_builder_check
iritkatriel Mar 2, 2023
| https://github.com/python/cpython/pull/102320/commits/fd7f2b3a4e3237fa502e28f6abc6f1209f27dae2 |
|
57fbe36
create shared helper function for resizing arrays
iritkatriel Mar 3, 2023
| https://github.com/python/cpython/pull/102320/commits/57fbe36db1cecf27762698cdb280b493f170ddc4 |
|
dc9f1f5
remove obsolete comment
iritkatriel Mar 3, 2023
| https://github.com/python/cpython/pull/102320/commits/dc9f1f51e927e3833848b9d4e52fd6d4f0d695e9 |
|
f20e480
Merge branch 'main' into instruction-stream
iritkatriel Mar 6, 2023
| https://github.com/python/cpython/pull/102320/commits/f20e4803436260d40f8613c7da7ea9b355e6d1b8 |
|
fa1c66a
address code review
iritkatriel Mar 7, 2023
| https://github.com/python/cpython/pull/102320/commits/fa1c66ab7b7805fd6bb035c352808371d00186f6 |
|
3528176
tweak comments
iritkatriel Mar 7, 2023
| https://github.com/python/cpython/pull/102320/commits/3528176db14f5ea8480d91b3c12eeb1a1f8121da |
|
fb13e36
index -> idx to avoid github's hilighting
iritkatriel Mar 7, 2023
| https://github.com/python/cpython/pull/102320/commits/fb13e36fbed8c66236ab6bdf584b5f421c23223b |
|
Clear filters
| https://github.com/python/cpython/pull/102320/files |
| Please reload this page | https://github.com/python/cpython/pull/102320/files |
| Please reload this page | https://github.com/python/cpython/pull/102320/files |
| Please reload this page | https://github.com/python/cpython/pull/102320/files |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |