Title: Remove cells_frees duplicate storage from Frame by youknowone · Pull Request #7351 · RustPython/RustPython · GitHub
Open Graph Title: Remove cells_frees duplicate storage from Frame by youknowone · Pull Request #7351 · RustPython/RustPython
X Title: Remove cells_frees duplicate storage from Frame by youknowone · Pull Request #7351 · RustPython/RustPython
Description: Cell/free variable objects were stored in both a separate Box<[PyCellRef]> (cells_frees field) and in the localsplus fastlocals array. Remove the redundant cells_frees field and access cell objects directly through localsplus, eliminating one Box allocation and N clone operations per frame creation. Summary by CodeRabbit Refactor Optimized VM frame variable storage mechanism by consolidating cell and free variable management for improved internal efficiency and code maintainability.
Open Graph Description: Cell/free variable objects were stored in both a separate Box<[PyCellRef]> (cells_frees field) and in the localsplus fastlocals array. Remove the redundant cells_frees field and access cell o...
X Description: Cell/free variable objects were stored in both a separate Box<[PyCellRef]> (cells_frees field) and in the localsplus fastlocals array. Remove the redundant cells_frees field and acces...
Opengraph URL: https://github.com/RustPython/RustPython/pull/7351
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:dff48b73-c8cf-00ee-4134-796db5016f87 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 89DA:1903FC:80051F:A9D076:6A5B9E5B |
| html-safe-nonce | 4f06773e8b99ac81a2b7b238c3d53aeb84b79b70d8fe981c2bee3f373673923a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4OURBOjE5MDNGQzo4MDA1MUY6QTlEMDc2OjZBNUI5RTVCIiwidmlzaXRvcl9pZCI6IjM0MzY5NDk4MjI2NTc3NjQ5NTYiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 2e9fd4f27317a228067570531d5b5229b4bdaba538078eab7f5d870e87af3a1a |
| hovercard-subject-tag | pull_request:3354269888 |
| 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 | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/RustPython/RustPython/pull/7351/files |
| twitter:image | https://avatars.githubusercontent.com/u/69878?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/69878?s=400&v=4 |
| og:image:alt | Cell/free variable objects were stored in both a separate Box<[PyCellRef]> (cells_frees field) and in the localsplus fastlocals array. Remove the redundant cells_frees field and access cell o... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5290d7e14309ad1e76106a9c4237bd1041517e83ea182c8ab756752cb0c6940b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/RustPython/RustPython git https://github.com/RustPython/RustPython.git |
| octolytics-dimension-user_id | 39710557 |
| octolytics-dimension-user_login | RustPython |
| octolytics-dimension-repository_id | 135201145 |
| octolytics-dimension-repository_nwo | RustPython/RustPython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 135201145 |
| octolytics-dimension-repository_network_root_nwo | RustPython/RustPython |
| 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 | 9c975978430e9ad293956f2bbdaf153b1bd84a99 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width