Skip to content

dis: Show names of intrinsics #103963

@JelleZijlstra

Description

@JelleZijlstra

For CALL_INTRINSIC_* opcodes, dis currently just shows the number, which makes it hard to figure out what the code actually does:

 2 LOAD_CONST 0 ('T') 4 CALL_INTRINSIC_1 7 

It would be nice if this instead said

 2 LOAD_CONST 0 ('T') 4 CALL_INTRINSIC_1 7 (INTRINSIC_TYPEVAR) 

Possible implementation strategy:

  • instrinsic names are defined in opcode.py
  • pycore_intrinsics.h is generated from opcode.py (bonus: we no longer have to manually update MAX_INTRINSIC_1)
  • dis learns to read the intrinsic names from opcode.py

Linked PRs

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.12only security fixesstdlibStandard Library Python modules in the Lib/ directorytype-featureA feature request or enhancement

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions