In function_module there is a check in infer_reuse_pattern, which is wrong. I…

In function_module there is a check in infer_reuse_pattern, which is wrong. I talked about it with James, he did a new version of infer_reuse_pattern and assumed that the new version should always return at least what the old version was returning. It turns out this is not the case and that the new version is correct.
上级 ed7f4669
...@@ -61,7 +61,7 @@ def infer_reuse_pattern(env, outputs_to_disown): ...@@ -61,7 +61,7 @@ def infer_reuse_pattern(env, outputs_to_disown):
# remove from rval all of the inputs, constants, values. # remove from rval all of the inputs, constants, values.
rval = set(r for r in rval if r.owner is not None) rval = set(r for r in rval if r.owner is not None)
if 1: if 0:
# DEBUG STUFF # DEBUG STUFF
# verify that we return a superset of what we've been returning so far... # verify that we return a superset of what we've been returning so far...
rval0 = _old_infer_reuse_pattern(env, outputs_to_disown) rval0 = _old_infer_reuse_pattern(env, outputs_to_disown)
......
Markdown 格式
0%
您添加了 0 到此讨论。请谨慎行事。
请先完成此评论的编辑!
注册 或者 后发表评论