[issue42093] Add opcode cache for LOAD_ATTR

STINNER Victor added the comment: New changeset d5fc99873769f0d0d5c5d5d99059177a75a4e46e by Victor Stinner in branch 'master': bpo-42093: Cleanup _PyDict_GetItemHint() (GH-24582) https://github.com/python/cpython/commit/d5fc99873769f0d0d5c5d5d99059177a75a4e46e -- __

[issue42093] Add opcode cache for LOAD_ATTR

Change by STINNER Victor : -- nosy: +vstinner nosy_count: 4.0 -> 5.0 pull_requests: +23361 pull_request: https://github.com/python/cpython/pull/24582 ___ Python tracker ___ ___

[issue42093] Add opcode cache for LOAD_ATTR

Guido van Rossum added the comment: Hm, I was thinking to recognize the specific type of descriptor used by slots and cache only that. Though we would still have to consider updates to C.__dict__ (that's handled by looking at the dict version right?). --

[issue42093] Add opcode cache for LOAD_ATTR

Pablo Galindo Salgado added the comment: > Thanks! Do you have any plans for further inline caches? Yeah, we are experimenting with some ideas here: https://bugs.python.org/issue42115. > I was wondering if we could reverse the situation for slots again by adding > slots support to the LOA

[issue42093] Add opcode cache for LOAD_ATTR

Guido van Rossum added the comment: Thanks! Do you have any plans for further inline caches? I was wondering if we could reverse the situation for slots again by adding slots support to the LOAD_ATTR opcode inline cache... -- nosy: +gvanrossum ___

[issue42093] Add opcode cache for LOAD_ATTR

Pablo Galindo Salgado added the comment: New changeset 9e8fe1986cb4205fb9f883c89b9d5d76a9847e0b by Pablo Galindo in branch 'master': bpo-42093: Tweak the what's new message about the new LOAD_ATTR opcode cache (GH-24070) https://github.com/python/cpython/commit/9e8fe1986cb4205fb9f883c89b9d5d

[issue42093] Add opcode cache for LOAD_ATTR

Change by Pablo Galindo Salgado : -- pull_requests: +22903 pull_request: https://github.com/python/cpython/pull/24070 ___ Python tracker ___ ___

[issue42093] Add opcode cache for LOAD_ATTR

Pablo Galindo Salgado added the comment: >Wow, this is amazing. I just found that this is now faster than slots. Not only that: is even faster than the highly-tuned namedtuple access descriptors that used to be the faster access to an attribute: 3.9 results --- 17.6 ns read_

[issue42093] Add opcode cache for LOAD_ATTR

Guido van Rossum added the comment: Wow, this is amazing. I just found that this is now faster than slots. Should we mention that in What's New? (Of course there's an optimization possible for slots as well, but it would require complicating the cache struct. Maybe in 3.11. :-) -- n

[issue42093] Add opcode cache for LOAD_ATTR

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue42093] Add opcode cache for LOAD_ATTR

Pablo Galindo Salgado added the comment: New changeset 109826c8508dd02e06ae0f1784f1d202495a8680 by Pablo Galindo in branch 'master': bpo-42093: Add opcode cache for LOAD_ATTR (GH-22803) https://github.com/python/cpython/commit/109826c8508dd02e06ae0f1784f1d202495a8680 -- ___

[issue42093] Add opcode cache for LOAD_ATTR

Change by Pablo Galindo Salgado : -- nosy: +yselivanov ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:

[issue42093] Add opcode cache for LOAD_ATTR

Change by Pablo Galindo Salgado : -- keywords: +patch pull_requests: +21760 stage: -> patch review pull_request: https://github.com/python/cpython/pull/22803 ___ Python tracker __

[issue42093] Add opcode cache for LOAD_ATTR

New submission from Pablo Galindo Salgado : >From the creators of "opcode cache for LOAD_GLOBAL" >(https://bugs.python.org/issue26219) now it's time for "opcode cache for >LOAD_ATTR: the revenge". This issue/PR builds on top of Yury's original patch >in the same way https://bugs.python.org/is