kparzysz-quic commented on pull request #8509: URL: https://github.com/apache/tvm/pull/8509#issuecomment-1050936801
We have a temporary workaround---just reset the `link_params` variable to `false` in fuse_ops, so the urgency for us is lower. There are two issues that need to be addressed: 1. The `link_params` property is inconsistent with the CPU target in `FuseOps`, when invoked through `FoldConstants`. 2. Float16 not being supported in AllocateConst visitors, e.g. https://github.com/apache/tvm/blob/main/src/relay/backend/te_compiler_cache.cc#L242 Hexagon does utilize the `LinkedParams` function, but we can change that as needed. So far, without any workarounds, we don't even get to the Hexagon codegen. -- 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]
