Repository navigation
Prepare naja-scope 0.1.19 with najaeda 0.7.29 - #27
Merged
Merged
Conversation
Upgrade to najaeda 0.7.29 and adopt its new raw-API features: - Follow the camelCase-only raw binding: LogicCone getNodeCount/getNodes/ getLeaves and naja.intentAvailable/intentTypeOf/intentParametersOf/ intentPackageMember (snake_case aliases were removed upstream). - Source references: read the language-neutral getSourceDeclaration / getSourceOrigins, falling back to getSourceLoc. Gate-level Verilog netlists now answer get_source, and extra origins are listed as `origins`. - load_verilog(yosys_source_attributes=True) decodes Yosys src attributes into RTL origins; get_source(path, origin=N) reads them. - load_vhdl accepts a list of files loaded as one batch in any order. - load_systemverilog(blackbox_multi_writer_memories=True) blackboxes modules with unsupported multi-writer memories instead of failing. - Revalidate the CORE-V-MCU demo baseline (631,603 leaves) and record the CVA6 v5.3.0 / master recheck in CLAUDE.md.
naja-schematic 0.2.0 requires najaeda>=0.7.29, drops the deprecated is_clock() call, and adds Show Sources; pass the session's source directories as its source_roots.
najaeda 0.7.29's memory inference no longer lets the cone run through the AXI adapter's registered (non-fall-through) I/D-cache data FIFOs. The fan-in of cva6.noc_req_o now stops at 15 registers inside cache_subsystem.i_adapter (16,145 nodes); 0.7.27 over-approximated it at ~2.47M nodes reaching back into the caches. Assert that the frontier lives in the adapter instead of requiring a >100k-node cone.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrade to najaeda 0.7.29 and adopt its new raw-API features:
getLeaves and naja.intentAvailable/intentTypeOf/intentParametersOf/
intentPackageMember (snake_case aliases were removed upstream).
getSourceOrigins, falling back to getSourceLoc. Gate-level Verilog netlists
now answer get_source, and extra origins are listed as
origins.into RTL origins; get_source(path, origin=N) reads them.
with unsupported multi-writer memories instead of failing.
CVA6 v5.3.0 / master recheck in CLAUDE.md.
Tests: 120 passed (incl. the 3 CVA6 regressions on a v5.3.0 snapshot rebuilt under 0.7.29); CORE-V-MCU demo passes.
🤖 Generated with Claude Code