GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — The inline cache for `LOAD_ATTR` is too large. · Issue #93533 · python/cpython

The inline cache for LOAD_ATTR is too large. #93533

New issue
Copy link
New issue
Copy link
Open
Open
The inline cache for LOAD_ATTR is too large.#93533
Copy link
Labels
3.12only security fixesonly security fixesinterpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)performancePerformance or resource usagePerformance or resource usage

Description

@markshannon
markshannon
opened on Jun 6, 2022
Issue body actions

The inline cache for LOAD_METHOD takes 10 code units, which is considerably larger than other inline caches, and is an obstacle to merging LOAD_ATTR and LOAD_METHOD (#93430), as LOAD_ATTR is such a common instruction.

See faster-cpython/ideas#396 for some ideas on how to shrink the inline cache.

Linked PRs

  • GH-93533: Shrink the LOAD_ATTR caches #103014

Activity

  1. markshannon
    added
    performancePerformance or resource usagePerformance or resource usage
    on Jun 6, 2022
  2. iritkatriel
    added
    3.12only security fixesonly security fixes
    on Sep 8, 2022
  3. markshannon
    changed the title [-]The inline cache for `LOAD_METHOD` is too large.[/-] [+]The inline cache for `LOAD_ATTR` is too large.[/+] on Oct 31, 2023

  4. markshannon commented on Oct 31, 2023

    @markshannon
    markshannon
    on Oct 31, 2023
    MemberAuthor
    More actions

    LOAD_METHOD has been merged into LOAD_ATTR, which means that the inline cache for LOAD_ATTR is too large.
    Changed the title reflect this.

  5. iritkatriel
    added
    interpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)
    on Nov 24, 2023
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

    3.12only security fixesonly security fixesinterpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)performancePerformance or resource usagePerformance or resource usage

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions
      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.