bespoke-silicon-group / bsg_manycore

Tile based architecture designed for computing efficiency, scalability and generality
Other
229 stars 58 forks source link

Add warning for unsupported atomic memory operations in cache adapters and infinite memory #412

Open drichmond opened 4 years ago

drichmond commented 4 years ago

https://github.com/bespoke-silicon-group/bsg_manycore/blob/91edf27e40d8dfb016029539084e324db190e99b/testbenches/common/v/bsg_nonsynth_mem_infinite.v#L205

When users run code with atomics on a design with inifite/ideal memory, the atomic operations will fail silently.

This can happen when a user is testing on one of the machines with a cache, and moves back to a fast 'n fake machine. Or, when the compiler inserts an unsupported atomic operation.

In either case, the code should print a warning.

drichmond commented 4 years ago

Slightly modified title. The same issue exists in the link-to-cache module

https://github.com/bespoke-silicon-group/bsg_manycore/blob/ff6668ddec0550fec52cc1dc7df6b48e23f8de62/v/bsg_manycore_link_to_cache.v#L248