search

pymtl / pymtl3

Pymtl 3 (Mamba), an open-source Python-based hardware generation, simulation, and verification framework
BSD 3-Clause "New" or "Revised" License
388 stars 46 forks source link

Translation/import bug fix #162

Closed ptpan closed 4 years ago

ptpan commented 4 years ago
  1. Fixed a bug where the Python import wrapper has incorrect connections for single element arrays
  2. Fixed a bug where the Python import wrapper has unresolved name references for nbits > 256
  3. Fixed a bug where nested arrays of nested interfaces are not correctly translated
  4. Removed the space between ; and the last word in the translation result
codecov[bot] commented 4 years ago

Codecov Report

Merging #162 into master will decrease coverage by <.01%. The diff coverage is 92.59%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
- Coverage   89.12%   89.11%   -0.01%     
==========================================
  Files         289      289              
  Lines       25013    25015       +2     
==========================================
+ Hits        22292    22293       +1     
- Misses       2721     2722       +1
Impacted Files Coverage Δ
.../backends/verilog/import_/test/VNameMangle_test.py 100% <ø> (ø) :arrow_up:
.../passes/backends/verilog/VerilogPlaceholderPass.py 84.12% <ø> (ø) :arrow_up:
.../translation/structural/VStructuralTranslatorL4.py 81.6% <0%> (-0.95%) :arrow_down:
pymtl3/passes/backends/yosys/util/utility.py 96.92% <100%> (ø) :arrow_up:
...es/backends/verilog/import_/VerilatorImportPass.py 93.23% <93.33%> (-0.23%) :arrow_down:
pymtl3/passes/backends/verilog/util/utility.py 89.36% <95.45%> (+0.39%) :arrow_up:
pymtl3/dsl/Connectable.py 83% <0%> (+0.27%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6d2cda6...a15c85c. Read the comment docs.