python / python/cpython

`LOAD_SMALL_INT` has empty `argrepr` via `Bytecode`

Aperta
#140,256 8 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

stdlib type-bug
Lingua principale
Python
Stelle
77.2k
Fork
35.9k
Metriche di merge delle PR
Metriche PR in attesa

Descrizione

Bug report

Bug description:

When using Bytecode to disassemble some Python code I've noticed that the argrepr attribute of the Instruction holding a LOAD_SMALL_INT isn't populated. For example:

Python 3.14.0 (main, Oct 16 2025, 09:17:39) [Clang 17.0.0 (clang-1700.3.19.1)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from dis import Bytecode
>>> list(Bytecode("1"))
[
    Instruction(
        opname='RESUME',
        opcode=128,
        arg=0,
        argval=0,
        argrepr='',
        offset=0,
        start_offset=0,
        starts_line=True,
        line_number=0,
        label=None,
        positions=Positions(lineno=0, end_lineno=1, col_offset=0, end_col_offset=0),
        cache_info=None
    ),
    Instruction(
        opname='LOAD_SMALL_INT',
        opcode=94,
        arg=1,
        argval=1,
        argrepr='',     # <--- Empty string
        offset=2,
        start_offset=2,
        starts_line=True,
        line_number=1,
        label=None,
        positions=Positions(lineno=1, end_lineno=1, col_offset=0, end_col_offset=1),
        cache_info=None
    ),
    Instruction(
        opname='RETURN_VALUE',
        opcode=35,
        arg=None,
        argval=None,
        argrepr='',
        offset=4,
        start_offset=4,
        starts_line=False,
        line_number=1,
        label=None,
        positions=Positions(lineno=1, end_lineno=1, col_offset=0, end_col_offset=1),
        cache_info=None
    )
]

I would have expected this to have the repr of the argument in there.

A quick and ignorant reading of dis.py suggests to me that there should be a change akin to this:

--- ../cpython/Lib/dis.py       2025-10-17 14:31:35.495151419 +0100
+++ my/dis.py   2025-10-17 14:43:26.650409153 +0100
@@ -586,7 +586,7 @@
             #    _disassemble_bytes needs the string repr of the
             #    raw name index for LOAD_GLOBAL, LOAD_CONST, etc.
             argval = arg
-            if deop in hasconst:
+            if deop in hasconst or deop == LOAD_SMALL_INT:
                 argval, argrepr = _get_const_info(deop, arg, self.co_consts)
             elif deop in hasname:
                 if deop == LOAD_GLOBAL:
CPython versions tested on:

3.14

Operating systems tested on:

macOS

Linked PRs
  • gh-140258

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia in Lib/dis.py, nella logica di gestione degli argomenti di Bytecode, e riproduci l’esempio con list(Bytecode("1")). Controlla il percorso LOAD_SMALL_INT e il comportamento di _get_const_info; il lavoro è completato quando il suo Instruction ha l’argrepr non vuoto previsto, con il comportamento verificato rispetto al lavoro collegato gh-140258.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
python
Ambito
devtools
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.