Title: Remove irregular stack effects · Issue #105678 · python/cpython · GitHub
Open Graph Title: Remove irregular stack effects · Issue #105678 · python/cpython
X Title: Remove irregular stack effects · Issue #105678 · python/cpython
Description: As we move towards generating more components from the instruction definition file (bytecodes.c), it helps if the instructions have regular formats and stack effects. Currently the stack effects can be simple, variable, conditional or co...
Open Graph Description: As we move towards generating more components from the instruction definition file (bytecodes.c), it helps if the instructions have regular formats and stack effects. Currently the stack effects ca...
X Description: As we move towards generating more components from the instruction definition file (bytecodes.c), it helps if the instructions have regular formats and stack effects. Currently the stack effects ca...
Opengraph URL: https://github.com/python/cpython/issues/105678
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Remove irregular stack effects","articleBody":"As we move towards generating more components from the instruction definition file (bytecodes.c), it helps if the instructions have regular formats and stack effects.\r\nCurrently the stack effects can be simple, variable, conditional or complex.\r\n* Simple: Does not depend on the operand (`oparg`)\r\n* Variable: Either pops or pushes `oparg * k` items.\r\n* Conditional: Pushes a value conditional on `oparg \u0026 1`\r\n* Complex: The stack effect depends on a set of flags embedded in `oparg`.\r\n\r\nWe can easily get rid of the complex case. There are only two cases, `MAKE_FUNCTION` and `FORMAT_VALUE`. Both can easily broken down into simple (and faster) parts.\r\n\r\nWe might want to get rid of condition stack effects, as it would simplify things, but performance may suffer as both `LOAD_GLOBAL` and `LOAD_ATTR` are conditional and they are performance critical.\r\n\r\n\r\n\u003c!-- gh-linked-prs --\u003e\r\n### Linked PRs\r\n* gh-105680\r\n* gh-105843\n\u003c!-- /gh-linked-prs --\u003e\r\n","author":{"url":"https://github.com/markshannon","@type":"Person","name":"markshannon"},"datePublished":"2023-06-12T09:21:17.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/105678/cpython/issues/105678"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:7ff41943-b1bd-0222-b578-99ca9e8b08ae |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 9A52:291671:BA5A8E:10129F4:6969F29A |
| html-safe-nonce | 74e361c8576223f0b0b6dd7dabb25a54076ce8b0afbecd1f173f4b1a5c70c34c |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5QTUyOjI5MTY3MTpCQTVBOEU6MTAxMjlGNDo2OTY5RjI5QSIsInZpc2l0b3JfaWQiOiIzNzk3NDM5NjI2MjI5NzA3NDE4IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 5e7704465def6ba6e73472b935134fb46059c10dbd6f64d4acf8230613d4588c |
| hovercard-subject-tag | issue:1752304101 |
| github-keyboard-shortcuts | repository,issues,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/_view_fragments/issues/show/python/cpython/105678/issue_layout |
| twitter:image | https://opengraph.githubassets.com/e9e629482c9e967f7b01c2ba90c9c7837f95766523e4bbbad7b880e40c69b1b1/python/cpython/issues/105678 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/e9e629482c9e967f7b01c2ba90c9c7837f95766523e4bbbad7b880e40c69b1b1/python/cpython/issues/105678 |
| og:image:alt | As we move towards generating more components from the instruction definition file (bytecodes.c), it helps if the instructions have regular formats and stack effects. Currently the stack effects ca... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | markshannon |
| hostname | github.com |
| expected-hostname | github.com |
| None | 7b32f1c7c4549428ee399213e8345494fc55b5637195d3fc5f493657579235e8 |
| turbo-cache-control | no-preview |
| 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 | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | bdde15ad1b403e23b08bbd89b53fbe6bdf688cad |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width