[GitHub] [tvm] masahi commented on pull request #12596: [TOPI][Bugfix] Make semantics of empty `axis` in `squeeze` consistent with Relay

2022-08-25 Thread GitBox
masahi commented on PR #12596: URL: https://github.com/apache/tvm/pull/12596#issuecomment-1227914137 hmm it's weird that `squeeze` is only tested on gpu targets: https://github.com/apache/tvm/pull/12596#L954 Can you add `llvm` there and replace `requires_gpu` with `uses_gpu`? --

[GitHub] [tvm] masahi commented on pull request #12596: [TOPI][Bugfix] Make semantics of empty `axis` in `squeeze` consistent with Relay

2022-08-25 Thread GitBox
masahi commented on PR #12596: URL: https://github.com/apache/tvm/pull/12596#issuecomment-1227735100 Can you add a test? -- 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