kpu/intgemm

Keep multiplication pure

Closed this issue · 4 comments

kpu commented

The multiply code should take a template argument for what it does with C. If there's a postprocess pipeline implemented using that template argument, fine. But the multiplier (i.e. multiply.h) should not impose that on users.

kpu commented

Cool. I think dvector_t is overkill. We could just have multiply call the callback twice. Or use std::pair.

Branch "callbacks and kernels" has been merged.

Do we want to do something more in this issue or we can close it?