stanford-ppl / spatial

Spatial: "Specify Parameterized Accelerators Through Inordinately Abstract Language"
https://spatial.stanford.edu
MIT License
274 stars 32 forks source link

Tweak Cost Metric for MemoryConfigurer #112

Closed mattfel1 closed 5 years ago

mattfel1 commented 6 years ago

I'm looking at a 2D memory with a 5x5 access parallel access, and the exhaustive banking found a hierarchical scheme N=5, alpha=List(1), B=1 | N=5, alpha=List(1),B=1 and another scheme N=64, alpha=List(1, 8), B=1. The current cost of depth * totalBanks is giving the hierarchical scheme, but now this means there are a bunch of mods and divides to handle the Ns = 5 which ends up being way more expensive.

Also I just realized this cost metric doesn't consider padding either right now. Will fix

mattfel1 commented 5 years ago

See issue #214