[GitHub] [tvm] llehtahw commented on pull request #7879: [Tensorize]Fix tensorize error while reusing compute

2021-04-21 Thread GitBox
llehtahw commented on pull request #7879: URL: https://github.com/apache/tvm/pull/7879#issuecomment-823957245 > why not use call_extern, etc. to achieve the same goal? In fact, my example was just drafted for showing the `Check failed`, so is't not important whether to use

[GitHub] [tvm] llehtahw commented on pull request #7879: [Tensorize]Fix tensorize error while reusing compute

2021-04-20 Thread GitBox
llehtahw commented on pull request #7879: URL: https://github.com/apache/tvm/pull/7879#issuecomment-823773672 Thank you @leeexyz . ``` _, _, intrin_c = get_compute_args() ``` That's also my workaround. I think both of the two usage should be considered valid. Would you

[GitHub] [tvm] llehtahw commented on pull request #7879: [Tensorize]Fix tensorize error while reusing compute

2021-04-19 Thread GitBox
llehtahw commented on pull request #7879: URL: https://github.com/apache/tvm/pull/7879#issuecomment-822935786 > This example seems not right to me. At the first glance, I don't think we should reuse the compute in this case. I agree with you. Not only this example, we are able to

[GitHub] [tvm] llehtahw commented on pull request #7879: [Tensorize]Fix tensorize error while reusing compute

2021-04-19 Thread GitBox
llehtahw commented on pull request #7879: URL: https://github.com/apache/tvm/pull/7879#issuecomment-822403781 @leeexyz @comaniac do you have any suggestion? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL