[quote="zhiics, post:15, topic:6334"] I have another thought on this, how about just put this one in the backend/utils.h since the current usage of them would be for the code under there? [/quote]
Yes, that's where I'd put this class in, given the current usage of `ExprFunctor`. [quote="tqchen, post:16, topic:6334"] As for the name of the `MemoizedExprFunctor` itself, I feel that the signature has divered from “functor” enough that perhaps we should chose another name. So that the “Functor” always takes a function signature as its template parameter. [/quote] Makes sense and `MemoizedExprTranslater<R>` sounds good to me. I'd change the typename to `OutputType` to make the meaning of type param obvious. --- [Visit Topic](https://discuss.tvm.ai/t/missing-memoization-in-exprfunctor/6334/17) 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/0cc53f9df3caa9f0ca46cae6d1d76d09bfdeb36200b0ea8e14c3ac2b80c21df8).
