Skip to content

Constant propagation for _BINARY_OP #115758

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Fidget-Spinner opened this issue Feb 21, 2024 · 2 comments
Open

Constant propagation for _BINARY_OP #115758

Fidget-Spinner opened this issue Feb 21, 2024 · 2 comments
Assignees
@Fidget-Spinner
Labels
type-feature A feature request or enhancement

Comments

@Fidget-Spinner
Copy link
Member

Fidget-Spinner commented Feb 21, 2024 •
edited by bedevere-app bot
Loading

Feature or enhancement

Proposal:

Parent issue #115506

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Linked PRs
@Fidget-Spinner Fidget-Spinner added the type-feature A feature request or enhancement label Feb 21, 2024
@Fidget-Spinner Fidget-Spinner mentioned this issue Feb 21, 2024
Open
2 tasks
@mdboom mdboom mentioned this issue Feb 28, 2024
Closed
32 tasks
@gvanrossum
Copy link
Member

FWIW, until we've implemented a mechanism that allows executors to own references to objects, we cannot replace e.g. LOAD_CONST 1000; LOAD_CONST 1000; BINARY_OP_ADD_INTwithNOP; NOP; LOAD_CONST_INLINE 1000000, because there's nothing to keep the inlined constants alive.

@Fidget-Spinner Fidget-Spinner self-assigned this Mar 30, 2024
@bedevere-app bedevere-app bot mentioned this issue Mar 30, 2024
Closed
@Fidget-Spinner
Copy link
Member Author

See faster-cpython/ideas#670 for design discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Fidget-Spinner Fidget-Spinner

Labels
type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gvanrossum @Fidget-Spinner