-
Notifications
You must be signed in to change notification settings - Fork 552
Changes to make CPU backend functions asynchronous calls #1174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This fixed `Assign.LinearAssignSeq` unit test in assign unit tests.
This also fixed assign unit test: `ArrayAssign.CPP_ASSIGN_VECTOR_2D`
Fixed lu async function
It was a bug in upstream function cpu::fast
|
build arrayfire ci |
|
build arrayfire windows ci |
|
@9prady9 Can you make sure Right now it fails only when trying to compile |
this macro takes care of asynchronous kernel launch and calls sync on the queue when in debug mode.
|
Code looks good. Things to change require the following:
|
|
@umar456 What do you think about the last point ? |
|
It would be nice to have a compile time option |
|
build arrayfire ci |
|
What would be the benefit of having both? Couldn't you just export the variable and make it persistent? |
|
@umar456 try setting and unsetting the variable on windows. It is not pretty. |
|
@9prady9 Can you pull from devel and resolve merge conflicts? You will need to fix |
|
@pavanky I will start on this after 3.2.2 -> devel PR. I believe that is the last hot fixes before v3.3.0 release, is that right ? |
|
@9prady9 Sure. |
This reverts commit 95d9346. * Not necessary with the new queue class (see queue.hpp) * Macros bad
|
build arrayfire ci |
Changes to make CPU backend functions asynchronous calls
Fixes #832