Lunderberg opened a new pull request, #16684:
URL: https://github.com/apache/tvm/pull/16684

   Prior to this commit, the `nn.modules.KVCache` implementations used 
`R.call_packed(...)` to call the `"vm.builtin.attention_*"` functions. Since 
`nn.Module` emits all relax functions within a `relax.DataflowBlock`, where 
impure expressions are forbidden, this is ill-formed.
   
   This commit updates the implementations in `nn.modules.KVCache` to use 
`R.call_pure_packed` instead of `R.call_packed`.  This assertation that the 
callee is pure allows the call to occur within a `relax.DataflowBlock`.


-- 
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]

Reply via email to