Avoid `exc=None; del exc` in bytecode where unnecessary
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 77.2k
- Forks
- 36k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
See https://github.com/faster-cpython/ideas/issues/490.
To break reference cycles, the compiler has an implicit del e at the end of an exception handler. To make UnboundLocalErrors impossible during that deletion, the compiler adds e=None; del e instead.
However, the e=None part is unnecessary in many situations, namely, when we can prove that e is guaranteed to be bound (almost always!).
This can reduce the size of some bytecode objects. For example, the four <---- lines below can be deleted:
>>> def f():
... try: pass
... except Exception as e: pass
...
>>> dis(f)
1 0 RESUME 0
2 2 LOAD_CONST 0 (None)
4 RETURN_VALUE
6 PUSH_EXC_INFO
3 8 LOAD_GLOBAL 0 (Exception)
20 CHECK_EXC_MATCH
22 POP_JUMP_IF_FALSE 11 (to 46)
24 STORE_FAST 0 (e)
26 POP_EXCEPT
28 LOAD_CONST 0 (None) <---------
30 STORE_FAST 0 (e) <---------
32 DELETE_FAST 0 (e)
34 LOAD_CONST 0 (None)
36 RETURN_VALUE
38 LOAD_CONST 0 (None) <---------
40 STORE_FAST 0 (e) <---------
42 DELETE_FAST 0 (e)
44 RERAISE 1
>> 46 RERAISE 0
>> 48 COPY 3
50 POP_EXCEPT
52 RERAISE 1
ExceptionTable:
6 to 24 -> 48 [1] lasti
38 to 46 -> 48 [1] lasti
- PR: gh-99361
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el issue 490 de faster-cpython/ideas y el PR gh-99361 enlazados, y después inspecciona la ruta del compilador de Python que emite bytecode de limpieza de los manejadores de excepciones. Usa el ejemplo dis(f) como referencia; se considera terminado cuando se omite la secuencia innecesaria exc=None; del exc allí donde se garantice que la variable de excepción está enlazada, sin cambiar la semántica de las excepciones.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- compilers
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 25/100