Skip to content
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.

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

[FIRRTL] Create debug info scopes for inlined modules #6512

Merged
merged 1 commit into from
Jan 22, 2024

Commits on Dec 13, 2023

  1. [FIRRTL] Create debug info scopes for inlined modules

    When inlining and flattening FIRRTL instances, create `dbg.scope`
    operations to track information about the original hierarchy in the
    debug info.
    
    To do this, the FIRRTL inliner now creates a `dbg.scope` op for every
    inlined instance. When renaming the operations in the inlined module's
    body, debug operations are assigned this scope (unless they already had
    a scope assigned). This retains information about the original
    hierarchy.
    
    Unfortunately, this approach currently only works for debug variables
    and scopes. Instances have no `scope` operand where a parent scope can
    be annotated. As a result, instance ops whose parent module got inlined
    do not properly track their original scope. This limitation will go away
    in the future once we are either able to interact with and modify the
    implicit scope created by instances, or instances scopes get passed in
    as explicit operands.
    fabianschuiki committed Dec 13, 2023
    Configuration menu
    Copy the full SHA
    6e30e31 View commit details
    Browse the repository at this point in the history