tqchen commented on pull request #7457:
URL: https://github.com/apache/tvm/pull/7457#issuecomment-780068579
I take another look and it seems that both vulkan and metal(after metal 2.2)
now support i64.
Perhaps we could update the ArgUnion solution of these APIs to allow pass 64
bit integer. For example
```c++
union ArgUnion64 {
int32_t v_int32;
uint32_t v_uint32;
float v_float32;
int64_t v_int64;
}'
```
On the device end, always create array of size two `int32_t v_int32[2]`, and
read the value from the first element. This would require all arguments to be
aligned to 64 bit, given not a lot of arguments are being passed in this way,
we should be good.
@masahi do you mind to make that change and test out instead?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
