Lunderberg commented on PR #14997: URL: https://github.com/apache/tvm/pull/14997#issuecomment-1570486985
Hmm. I think it might be related to [this change](https://github.com/apache/tvm/pull/14918/files#diff-c795c8cdf29dcfda3dcb7bf342247b22deee71e2a9c037046254625e91bdd152L264) in particular. Previously, the host function had its `kTarget` attribute removed altogether. Afterwards, the host function had its `kTarget` attribute updated to be the original functions `target->GetHost()`. Effectively, that would result in any per-function host annotations being replaced with the host passed to `tvm.build`, which defaults to LLVM. That still wouldn't explain how the LLVM intrinsics were inserted in the first place, as `LowerIntrin` should use whatever the function annotation is at the point of the pass being applied, but it might be close to an answer. -- 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]
