[GitHub] [tvm] janetsc commented on pull request #12957: [Hexagon] Remove TVM logging from performance-sensitive parts of runtime

2022-10-05 Thread GitBox
janetsc commented on PR #12957: URL: https://github.com/apache/tvm/pull/12957#issuecomment-1265677915 > Ah, I see some tests are throwing this even for runtime_hexbuffs (before ReleaseResources is called.). I'm going to continue to investigate. -- This is an automated message from

[GitHub] [tvm] janetsc commented on pull request #12957: [Hexagon] Remove TVM logging from performance-sensitive parts of runtime

2022-10-05 Thread GitBox
janetsc commented on PR #12957: URL: https://github.com/apache/tvm/pull/12957#issuecomment-1265594905 > The degradation started a while back, but we didn't notice, because the tests were passing without timing out. The main contributor to the degradation is the exception being thrown, but I

[GitHub] [tvm] janetsc commented on pull request #12957: [Hexagon] Remove TVM logging from performance-sensitive parts of runtime

2022-10-05 Thread GitBox
janetsc commented on PR #12957: URL: https://github.com/apache/tvm/pull/12957#issuecomment-1265527449 > > Our local simulator tests would time out after 1h, now they finish within 30 min. I used a subset of the fp16 conv tests, and without this change they would take about 4