Yeah, I am not a big fun of introducing this base class either as I think the only duplication code would be really just the caching map. If you are concerning about that 10 locs. I can actually just do it this way, I can actually remove them and replace it by calling the Functor<R(Expr)>::VisitExpr(expr); I did it that way because I wanted to give an error for unsupported node. Alternatively, I can have a checker for that.
--- [Visit Topic](https://discuss.tvm.ai/t/missing-memoization-in-exprfunctor/6334/9) to respond. You are receiving this because you enabled mailing list mode. To unsubscribe from these emails, [click here](https://discuss.tvm.ai/email/unsubscribe/b331cfcbe3709c4a982e7330fdbf6ed12365c46eb72e73f7e36e1bdcaa920105).
