areusch commented on issue #8953: URL: https://github.com/apache/tvm/issues/8953#issuecomment-924495223
gotcha--okay so we are just exercising these models with AOT at the present moment. I think reducing kMaxFuncNameLength is not a great option because it will constrain all other use cases just to make the C runtime use case work. as an aside for microTVM we are generally in favor of deprecating the GraphExecutor as soon as AOT reaches parity. i'm okay with making the `func_name` array length configurable in `crt_config.h`. i think that would address the issue without requiring more RAM for all GraphExecutor use cases. do you want to submit a PR for this? -- 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: commits-unsubscr...@tvm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
