class Memory: allow to set default simulate value for all generated memory blocks.

[nmigen-issue-migration]

Issue by Fatsie
Tuesday Nov 12, 2019 at 13:43 GMT
Originally opened as m-labs/nmigen#268

---

I have nmigen code mixed with other RTL language code so am using external simulator. With this option I can now turn off the memory preparation for pysim for all generated blocks.

from nmigen import *

Memory.simulate = False

---

Fatsie included the following code: https://github.com/m-labs/nmigen/pull/268/commits

[nmigen-issue-migration]

Comment by codecov[bot]
Tuesday Nov 12, 2019 at 13:58 GMT

---

Codecov Report

Merging #268 into master will decrease coverage by <.01%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   82.07%   82.07%   -0.01%     
==========================================
  Files          34       34              
  Lines        5597     5600       +3     
  Branches     1200     1201       +1     
==========================================
+ Hits         4594     4596       +2     
  Misses        863      863              
- Partials      140      141       +1

Impacted Files	Coverage Δ
nmigen/hdl/mem.py	97.22% <75%> (-0.88%)	⬇️

---

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 f8f7d83...40884c9. Read the comment docs.

[nmigen-issue-migration]

Comment by whitequark
Friday Nov 15, 2019 at 23:29 GMT

---

I'm rewriting the simulator right now so that it would not need the (expensive) memory lowering. So I'll keep this open, with the understanding that Memory.simulate will go away completely.