arnavsacheti / PeakRDL-BusDecoder / 23727563641

92%

Build Type

push

github

Committed by web-flow

Commit Message

Add SystemRDL parameter monkeypatch to expose root addrmap params in BusDecoder RTL (#54)

* Add SystemRDL parameter monkeypatch to expose root addrmap params in BusDecoder RTL

Monkeypatches ParameterRef.get_value() to trace which root-level addrmap
parameters are referenced throughout the component tree, classifying each as:

- DIRECT: non-address-affecting params passed through as SV `parameter` on
  the generated module (e.g. reset values, field widths)
- ADDRESS_MODIFYING: params that control array dimensions become SV
  `parameter` with runtime constraint n <= N. The decoder generates the full
  address space for N (elaborated max), but for-loop bounds use the parameter
  name so the RTL can reduce the active element count at instantiation time.

Key changes:
- New rdl_params.py: RdlParameterExtractor, RdlParameter, ParameterUsage
- DesignState: extracts/classifies params, provides enable lookup per dimension
- ForLoopBody: accepts int|str for parameterized loop bounds
- decode_logic_gen, fanout_gen, fanin_gen, fanin_intermediate_gen: use
  parameter names as loop bounds for enable-gated array dimensions
- module_tmpl.sv: adds initial-block assertions for n <= N constraints
- package_tmpl.sv: emits MAX_<PARAM> constants
- base_cpuif: skips redundant localparam when RDL param covers the array

https://claude.ai/code/session_01G5LEnnwCck4X4tjN1vQAbd

* Only expose address-modifying parameters in the decoder

Non-address parameters (reset values, field widths, etc.) are irrelevant
to the bus decoder and are now silently ignored instead of being passed
through as SV parameters.

https://claude.ai/code/session_01G5LEnnwCck4X4tjN1vQAbd

* Add documentation for SystemRDL parameter handling

Explains which root-level addrmap parameters are extracted (address-modifying
only), how the monkeypatch tracing detects them, and how they appear in the
generated RTL (module params, assertions, loop bounds, package MAX constants).

https://claude.ai/code/session_0... (continued)

Coverage Stats

176 of 206 new or added lines in 10 files covered. (85.44%)

18 existing lines in 5 files now uncovered.

1397 of 1522 relevant lines covered (91.79%)

0.92 hits per line