tqchen opened a new pull request, #19528: URL: https://github.com/apache/tvm/pull/19528
## Summary `AttrFunctor` is declared infrastructure with no remaining users. An exhaustive search across `include/`, `src/`, `tests/`, `python/`, `apps/`, `web/`, and `cmake/` confirms zero subclasses, zero friend declarations, and zero macro callers outside the header itself — its two internal macros (`ATTR_FUNCTOR_DEFAULT`, `ATTR_FUNCTOR_DISPATCH`) are only used inside `src/ir/attr_functor.h`. The one `#include "attr_functor.h"` in `src/ir/attrs.cc` is a stale leftover from a prior migration; that file references only `DictAttrs` and `AttrFieldInfoNode` (from `tvm/ir/attrs.h`), not any `AttrFunctor` symbols. Removes `src/ir/attr_functor.h` (150 lines) and drops the stale include. No build-system changes needed — the header was never enumerated in any `CMakeLists.txt`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
