Commit afa20869 by Tianqi Chen Committed by GitHub

[PASS] More improvement of canonical (#314)

parent 3c2569a0
...@@ -191,6 +191,9 @@ class Canonical::Internal : public IRMutator { ...@@ -191,6 +191,9 @@ class Canonical::Internal : public IRMutator {
ret_entry_.max_level = stack_.back().max_level; ret_entry_.max_level = stack_.back().max_level;
stack_.pop_back(); stack_.pop_back();
CHECK(expr.defined()); CHECK(expr.defined());
if (const IntImm* op = expr.as<IntImm>()) {
return Mutate_(op, expr);
}
return expr; return expr;
} }
// call produce to get a cache entry. // call produce to get a cache entry.
......
...@@ -27,6 +27,16 @@ def test_basic(): ...@@ -27,6 +27,16 @@ def test_basic():
assert str(ret.value) == "(m - 1)" assert str(ret.value) == "(m - 1)"
def test_canonical():
x = tvm.var("x")
z = tvm.const(3)
ret = tvm.ir_pass.CanonicalSimplify(x / (z*z) - x / (z*z))
assert(tvm.ir_pass.Equal(ret, 0))
ret = tvm.ir_pass.CanonicalSimplify(x / (z+z) - x / (z+z))
assert(tvm.ir_pass.Equal(ret, 0))
if __name__ == "__main__": if __name__ == "__main__":
test_basic() test_basic()
test_simplify() test_simplify()
test_canonical()
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment