Andrew T. Barker
Andrew T. Barker
I have a smaller reproducer, unfortunately it's not very informative to me. Here's the source: ```c++ #include #include "oneapi/mkl.hpp" int main(int argc, char * argv[]) { std::cout
Replaced by #227.
@mkrainiuk This sounds like the right solution to me. Let me know how I can help.
After some more investigation of this connected to #210, @mmeterel and I notice this affects tests that call cblas interface to oneMKL but not Fortran interface. Unfortunately in #210 we...
@aaronjohnson I addressed your comments and added the table you suggested, let me know what you think.
@mmeterel we have a TAB meeting on these changes next week, I think it makes sense to wait until after the meeting to merge.
@aaronjohnson @mmeterel @lhuot @spencerpatty I've modified the spec based on feedback from TAB and others: 1. Added documentation of in-place use of `omatadd` to do `C += alpha A` and...
In my view this is ready to merge - let me know if you have any concerns.
@spencerpatty I made most of the changes you suggested (add "dense" and use "specify" instead of "implement") in several places. Regarding transpose options: I clarified the transpose datatype description [here](https://github.com/oneapi-src/oneAPI-spec/pull/420/files#diff-5202a2c59db032e3de63e3cf1f485d9001284146663a60547e1c11ee5510b4a5),...
Can you share your motivation for introducing the `dense_matrix_handle_t` type? The dense BLAS parts of the oneAPI spec do not use such a handle and I'm not sure it's helpful...