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] Use untyped propassign source accessor in LowerClasses. #6690

Merged
merged 1 commit into from
Feb 15, 2024

Commits on Feb 13, 2024

  1. [FIRRTL] Use untyped propassign source accessor in LowerClasses.

    When lowering object instantiations, we need to get the source from a
    propassign to pass that Value to the newly created OM dialect object
    op. However, in some cases, like when passing references to objects
    down the hierarchy, we have already converted the source Value to an
    OM dialect object op. This means the typed getSrc accessor will fail,
    as it expects the source Value to be a FIRRTLType.
    
    The IR is in the desired state at this point, so instead we can simply
    use the untyped getSrcMutable accessor to get the corresponding
    OpOperand, and get the Value from there. This avoids errors when the
    pass has knowingly put the IR into a state that won't pass the
    verifier.
    mikeurbach committed Feb 13, 2024
    Configuration menu
    Copy the full SHA
    8202952 View commit details
    Browse the repository at this point in the history