areusch commented on PR #11105: URL: https://github.com/apache/tvm/pull/11105#issuecomment-1109168340
thanks @kparzysz-quic for this PR. just a couple thoughts: - i think these are all different renderings of a single runtime::Module produced by the `llvm` codegen iiuc. could we potentially avoid adding flags to Target by using `Module::Save`? - in the distant future when we might have that [Artifact](https://discuss.tvm.apache.org/t/introduce-artifact-a-container-for-generated-code/11099) refactor, it seems like we could allow a codegen to emit multiple artifacts and not include all of them in an exported library by marking such metadata. just wanted to get your thoughts whether that flow might be compatible with the intended use case for this PR? -- 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]
