Question about registerfile about vicuna HOT 2 CLOSED

Hi @moimfeld , thank you very much for the work you are doing!

The XORs are only required for the correct functionality of the XOR RAM as described here: https://tomverbeure.github.io/2019/08/03/Multiport-Memories.html#xor-based-multi-port-rams

If you are implementing a RAM that can support multiple write ports without requiring that the entire register file is replicated for each write port, then these are not required. An important feature that the register file must support is that register data can be written at byte granularity, based on the byte enable signal, but I see that you have implemented that in your ASIC version.

Please let me know if you have any other questions or if there is anything that you need or things that you would like to discuss!

from vicuna.

Hi @michael-platzer

Thanks a lot for your answer, this resolves my question.
I will open a pull-request once the Questasim environment has passed all tests. There will be some small RTL changes that need to be discussed once I open the PR (just for your information). But for now everything is clear as your code is very well structured and easy to understand.

from vicuna.