Skip to content

Move duplicate LOAD_SMALL_INT optimization from codegen to CFG #130480

Closed
Closed
Move duplicate LOAD_SMALL_INT optimization from codegen to CFG#130480
Labels
interpreter-core(Objects, Python, Grammar, and Parser dirs)type-featureA feature request or enhancement

Description

@WolframAlph
WolframAlph
opened on Feb 23, 2025

Feature or enhancement

Proposal:

Addressing #129568 (comment)

Currently both codegen & peepholer optimize LOAD_CONST for small integers to LOAD_SMALL_INT. CFG can do it alone, so we don't have this logic in 2 places.

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Linked PRs

Activity

  1. WolframAlph
    added
    type-featureA feature request or enhancement
    on Feb 23, 2025
  2. picnixz
    added
    interpreter-core(Objects, Python, Grammar, and Parser dirs)
    on Feb 23, 2025
  3. WolframAlph
    changed the title [-]Move `LOAD_SMALL_INT` optimization from codegen to CFG[/-] [+]Move duplicate `LOAD_SMALL_INT` optimization from codegen to CFG[/+] on Mar 13, 2025
  4. iritkatriel
    added a commit that references this issue on Mar 14, 2025

    gh-130480: Move duplicate `LOAD_SMALL_INT` optimization from codegen …

    55815a6
  5. WolframAlph
    closed this as completedon Mar 14, 2025
  6. plashchynski
    added a commit that references this issue on Mar 17, 2025

    pythongh-130480: Move duplicate `LOAD_SMALL_INT` optimization from co…

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    interpreter-core(Objects, Python, Grammar, and Parser dirs)type-featureA feature request or enhancement

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions