Thanks for continuing to give feedback @lebeg. The pros I see for that approach: * Handles OOB * Works for various dest tensor types * Casts the void pointer to the correct type and assigns, which I think should be the correct way to deal with this.
Cons: * Not super concise and a little difficult to read. I think I'm still slightly in favour of the current fix. Not sure if this is perf critical code so my bias would be towards readability and correctness here. This is my weak preference though, I'd also be fine with your approach. [ Full content available at: https://github.com/apache/incubator-mxnet/pull/12631 ] This message was relayed via gitbox.apache.org for [email protected]
