Skip to content

bpo-43693: Compute deref offsets in compiler #25152

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

Merged
ericsnowcurrently merged 12 commits into from
Jun 4, 2021
Merged

bpo-43693: Compute deref offsets in compiler #25152

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
d2f617f
Compute index of cellvars in compiler to avoid additional pointer in …
markshannon Jan 7, 2021
4893684
Remove LOAD_CLOSURE instruction.
markshannon Mar 31, 2021
2f32044
Add NEWS
markshannon Apr 2, 2021
0db1edc
Bump the bytecode magic number.
ericsnowcurrently Apr 24, 2021
4d82f15
Update the dis module docs.
ericsnowcurrently Apr 24, 2021
e4bf977
Clarify the LOAD_FAST docs.
ericsnowcurrently Apr 26, 2021
d25324a
Update the *_DEREF docs.
ericsnowcurrently Apr 26, 2021
775d4db
Preserve the LOAD_CLOSURE opcode as an alias for LOAD_FAST.
ericsnowcurrently Apr 26, 2021
5183325
Update the dis docs.
ericsnowcurrently May 27, 2021
4f1dde5
touch
ericsnowcurrently Jun 3, 2021
46d531e
touch
ericsnowcurrently Jun 3, 2021
eafb2b1
Drop the "cell" arg from code._varname_from_oparg().
ericsnowcurrently Jun 3, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update the dis docs.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Jun 3, 2021
commit 5183325f3f1c05aede20e4677e77302a20fc4315
35 changes: 16 additions & 19 deletions Doc/library/dis.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1058,25 +1058,23 @@ All of the following opcodes use their arguments.

.. opcode:: LOAD_CLOSURE (i)

Pushes a reference to the cell contained in slot ``i - len(co_varnames)``
of the cell and free variable storage. The name of the variable is
``(co_varnames + co_cellvars + co_freevars)[i]``.
Pushes a reference to the cell contained in slot ``i`` of the "fast locals"
storage. The name of the variable is ``co_fastlocalnames[i]``.

Note that ``LOAD_CLOSURE`` is effectively an alias for ``LOAD_FAST``.
It exists to keep bytecode a little more readable.

.. versionchanged:: 3.10
``i`` is offset by the length of ``co_varnames``.
.. versionchanged:: 3.11
``i`` is no longer offset by the length of ``co_varnames``.


.. opcode:: LOAD_DEREF (i)

Loads the cell contained in slot ``i - len(co_varnames)`` of the cell
and free variable storage.
Loads the cell contained in slot ``i`` of the "fast locals" storage.
Pushes a reference to the object the cell contains on the stack.

.. versionchanged:: 3.10
``i`` is offset by the length of ``co_varnames``.
.. versionchanged:: 3.11
``i`` is no longer offset by the length of ``co_varnames``.


.. opcode:: LOAD_CLASSDEREF (i)
Expand All @@ -1087,29 +1085,28 @@ All of the following opcodes use their arguments.

.. versionadded:: 3.4

.. versionchanged:: 3.10
``i`` is offset by the length of ``co_varnames``.
.. versionchanged:: 3.11
``i`` is no longer offset by the length of ``co_varnames``.


.. opcode:: STORE_DEREF (i)

Stores TOS into the cell contained in slot ``i - len(co_varnames)``
of the cell and free variable storage.
Stores TOS into the cell contained in slot ``i`` of the "fast locals"
storage.

.. versionchanged:: 3.10
``i`` is offset by the length of ``co_varnames``.
.. versionchanged:: 3.11
``i`` is no longer offset by the length of ``co_varnames``.


.. opcode:: DELETE_DEREF (i)

Empties the cell contained in slot ``i - len(co_varnames)``
of the cell and free variable storage.
Empties the cell contained in slot ``i`` of the "fast locals" storage.
Used by the :keyword:`del` statement.

.. versionadded:: 3.2

.. versionchanged:: 3.10
``i`` is offset by the length of ``co_varnames``.
.. versionchanged:: 3.11
``i`` is no longer offset by the length of ``co_varnames``.


.. opcode:: RAISE_VARARGS (argc)
Expand Down