The fact that you even can .store_root().store_in(Register) is bananas. The fact that Halide compiles this as a sliding window optimization without actually store_root() should be documented somewhere. This feels closer to fold_storage() (which it isn't because it's sliding, not looping), than to store_root().
Just dropping this here, as I didn't know that, and discovered that while debugging correctness/sliding_window for some PR. Assigning @abadams because I believe you wrote the sliding window stuff and the accompanying tests. You'll know best what to do.
The fact that you even can
.store_root().store_in(Register)is bananas. The fact that Halide compiles this as a sliding window optimization without actually store_root() should be documented somewhere. This feels closer tofold_storage()(which it isn't because it's sliding, not looping), than tostore_root().Just dropping this here, as I didn't know that, and discovered that while debugging correctness/sliding_window for some PR. Assigning @abadams because I believe you wrote the sliding window stuff and the accompanying tests. You'll know best what to do.