Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support for evaluating multiple outputs simultaneously #1462

Merged
merged 16 commits into from
Jun 17, 2016
Merged

Support for evaluating multiple outputs simultaneously #1462

merged 16 commits into from
Jun 17, 2016

Conversation

pavanky
Copy link
Member

@pavanky pavanky commented Jun 17, 2016 •
edited
Loading

@pavanky
OpenCL: Modifying evalNodes to handle multiple outputs
f03f734
@pavanky
FEAT: Adding API evaluating multiple outputs
c5e4456
@pavanky
Fixes to remove duplicate BufferNodes from AST
3247988 
- This has side affect on reference counts.
@pavanky
More fixes to multi eval in OpenCL
312eca9
@pavanky
PERF: Remove redundant traversals of the JIT Trees
e4f93f6
@pavanky
Changing af::eval() to use pointers
cdc27cd
@pavanky
FEAT: Adding flags to set internal eval flags
86da253
@pavanky
BUGFIX: Fix to multi_eval in opencl backend
941d4b5
@pavanky
Fix for missing symbols in unified backend
fe08715
@pavanky
BUGFIX: Do not error out when all inputs are evaluated
7aa8e7b
@pavanky
Fixes to OpenCL JIT for buffer nodes
6d0d62b
@pavanky
Cleaning up BufferNode creation in OpenCL backend
83ea446
@pavanky
Fixes to reuse buffer nodes for CUDA backend
bccac45
@pavanky
Adding multi_eval functionality for CUDA backend
7dd2a6d
@pavanky
TEST: Adding tests for multi_eval
3486817
@pavanky
Renaming functions to control manualEvalFalgs
6f85985
@pavanky pavanky added this to the 3.4.0 milestone Jun 17, 2016
umar456
umar456 reviewed Jun 17, 2016
View reviewed changes
src/backend/cuda/platform.cpp
@@ -488,6 +488,12 @@ bool synchronize_calls() {
return sync;
}

bool& evalFlag()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't need to be a reference does it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function contains a static variable. I am using the function to set it.

@umar456 umar456 merged commit 80f2e64 into arrayfire:devel Jun 17, 2016
@pavanky pavanky deleted the multi_eval branch June 17, 2016 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
3.4.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@pavanky @umar456