Skip to content

Move check on too large inferred memories to immediately after canonicalization #3650

@oharboe

Description

@oharboe

Description

Earlier failure is better. For #3645, the error would appear after 10 seconds instead of 15 minutes.

Also, can the code be simplified to query the Yosys directly instead going via the Python script to parse the .json file?

json -o $::env(RESULTS_DIR)/mem.json
# Run report and check here so as to fail early if this synthesis run is doomed
exec -- $::env(PYTHON_EXE) $::env(SCRIPTS_DIR)/mem_dump.py \
--max-bits $::env(SYNTH_MEMORY_MAX_BITS) $::env(RESULTS_DIR)/mem.json

Suggested Solution

No response

Additional Context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions