Lunderberg commented on PR #16379: URL: https://github.com/apache/tvm/pull/16379#issuecomment-1908257983
Ah, thank you. I had interpreted not having "explicit `sinfo_args`" as having a null pointer for the `Array<StructInfo> sinfo_args`, and wanted to check if that would instead indicate a bug in `CallTIRRewrite`. Instead, `CallTIRRewrite` produces a well-defined, but empty, `sinfo_args`. The mismatch is in the TVMScript printer, which suppresses printing of empty `sinfo_args`, and the TVMScript parser, which currently requires `sinfo_args` even if empty. Thank you for clearing up my confusion. -- 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]
