Mousius commented on pull request #8917: URL: https://github.com/apache/tvm/pull/8917#issuecomment-913114511
> I'm not sure if it's a good idea to have a test case that attempts to cause stack overflow without this PR. That test case may be flaky due to different stack sizes on different CI machines. Sorry, I meant for it to be included in this PR so we have test coverage and some way of tracking if this regresses. I understand it might not always pick up the issue on all machines but it shouldn't fail if this change is in place? -- 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]
