mattfel1

The problem here is that there is a RegRead that has one instance after unrolling. Specifically, it is used for the start of a counter in a MemReduce. After unrolling,...

Much easier app to test with: ``` @spatial class MemReduce1D extends SpatialTest { override def runtimeArgs: Args = "1920" val tileSize = 64 val p = 1 def blockreduce_1d[T:Num](src: Array[T],...

I put a little patch in BufferRecompute that we should get rid of once we have a real fix for this

Right now the solution is to just have a unique val for each instance where you use that val.

Hi. Thanks so much for the help! I think this optimization will have a big impact on a lot of apps. I'm looking forward to seeing how it goes. For...

For Const, we often do things like `x match {case Const(c:Int) => }`. For other situations, if you do `bound(x) = 512` in an app, it sets the metadata `x.bound`...

After thinking about this more, I don't think its something we should implement right now until more people request it.

We will definitely start from there. I think area estimation pre-unrolling/mem analysis is going to be tricky now since small changes in parallelizations and dse parameters can have huge impacts...

Got it. More concerning is that --synth and --sim don't match up in this case. One or both are doing something suspicious, but either way I will go in and...

This looks like there are old packages hanging around. Can you try deleting these directories to get a totally fresh start and trying again? ~/.ivy2/cache/edu.stanford.cs.dawn ~/.ivy2/local/edu.stanford.cs.dawn ~/spatial-quickstart/target ~/spatial-quickstart/project/target ~/spatial-quickstart/gen ?