tqchen commented on PR #16769:
URL: https://github.com/apache/tvm/pull/16769#issuecomment-2015547796
agree that we should have a carved out testcases here, will followup in
comment soon, as well as in the original thread
I believe it was along the original lines of comment in the changed order,
which i assume we say that we cannot merge dyn smem beyond the kernel
boundary,
```c++
// MergeSharedMemoryAllocations must be applied after SplitHostDevice
// because the merged allocation site is at the beginning of each device
function
mixed_pass_list.push_back(tir::transform::MergeSharedMemoryAllocations());
```
--
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]
