Skip to content

Bytecodes cleanup: promote TIER_{ONE,TWO}_ONLY to properties #115778

Closed

Description

@gvanrossum

Instead of

op(FOO, (left, right -- res)) {
    TIER_TWO_ONLY
    <actual code>
}

I'd like to be able to write

tier2 op(FOO, (left, right -- res)) {
    <actual code>
}

Linked PRs

Activity

  1. gvanrossum commented on Feb 21, 2024

    @gvanrossum
    MemberAuthor

    If someone wants to pick this up I'm happy to provide some clues as to where to make changes (but multi-file grep goes a long way. :-)

  2. Eclips4 commented on Feb 21, 2024

    @Eclips4
    Member

    I would like to pick this up if you don't mind :)

  3. gvanrossum commented on Feb 21, 2024

    @gvanrossum
    MemberAuthor

    @Eclips4 Go for it! Remember to run make regen-cases frequently (after each change to the generator and each change to bytecodes.c or the redundancy eliminator input file).

  4. self-assigned this
    on Feb 21, 2024
  5. added a commit that references this issue on Feb 22, 2024
  6. gvanrossum commented on Feb 23, 2024

    @gvanrossum
    MemberAuthor

    For the CI failures, see gh-115859. Once that's fixed, please merge the latest main and push again.

  7. added a commit that references this issue on Feb 23, 2024
  8. added a commit that references this issue on Mar 4, 2024
  9. added a commit that references this issue on Apr 17, 2024
  10. added a commit that references this issue on Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

easyinterpreter-core(Objects, Python, Grammar, and Parser dirs)

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions