tqchen commented on PR #15835: URL: https://github.com/apache/tvm/pull/15835#issuecomment-1789308138
Looking at the error. This particular issue is due to the fact that metal codegen will construct a structural buffer to pass all the arguments. Right now normal codegen requires first declare then define the functions, resulting in the same struct definition to appear twice. WebGPU likely had a similar issue so we simply rolled to the original implementations in the unity branch. We do not need to handle cross function in GPU shaders, so one simple approach should be simply skip the declare steps -- 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]
