You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
diff --git a/builder/simplify.cc b/builder/simplify.cc
index aeaa156..84e9555 100644
--- a/builder/simplify.cc
+++ b/builder/simplify.cc
@@ -1239,7 +1239,7 @@ public:
if (auto cstep = as_constant(step)) alignment.modulus = *cstep;
if (auto cmin = as_constant(bounds.min)) alignment.remainder = *cmin;
// If we're in the body of the loop, then we know that bounds.max >= bounds.min.
- stmt body = mutate_with_bounds(op->body, op->sym, {bounds.min, mutate(max(bounds.min, bounds.max))}, alignment);
+ stmt body = mutate_with_bounds(op->body, op->sym, {bounds.min, bounds.max}, alignment);
for (auto& i : buffers) {
if (i) --i->loop_depth;
}
This change slightly regressed the simplification of this example:
SLINKY_VERBOSE=1 bazel run --config=ubsan builder/test:pipeline -- --gtest_filter=split_schedule_mode/elementwise.pipeline_2d/2147483647_1_0
#560 has a change reverted by the following diff:
This change slightly regressed the simplification of this example:
from producing:
vs.
Note the extra nesting of the
intm.out.x = crop_dim(...)
.The text was updated successfully, but these errors were encountered: