-
Notifications
You must be signed in to change notification settings - Fork 10
Update all dependencies #11
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -104,7 +104,7 @@ SMI_Comm SmiInit_{{ name }}( | |
| const int num_kernels = kernel_names.size(); | ||
| for (int i = num_kernels - 1; i >= 0; i--) | ||
| { | ||
| queues[i].enqueueTask(kernels[i]); | ||
| queues[i].enqueueNDRangeKernel(kernels[i], cl::NullRange, cl::NDRange(1)); | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there any advantage in using
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. enqueueTask is no more available in cl2 |
||
| queues[i].flush(); | ||
| } | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,6 +15,11 @@ | |
| #include <utils/utils.hpp> | ||
| #define TILE_SIZE 128 //define this as used in the opencl kernel | ||
|
|
||
| #if !defined(CL_CHANNEL_1_INTELFPGA) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think that this should not be necessary |
||
| // include this header if channel macros are not defined in cl.hpp (versions >=19.0) | ||
| #include "CL/cl_ext_intelfpga.h" | ||
| #endif | ||
|
|
||
| using namespace std; | ||
| float *A,*B,*x,*y; | ||
| float *fpga_res_y; | ||
|
|
@@ -150,7 +155,7 @@ void testStreamed(std::string program_path,int n, int m, float alpha, float beta | |
| comp_start=current_time_usecs(); | ||
| asm volatile("": : :"memory"); | ||
| for(int i=0;i<kernel_names.size();i++) | ||
| queues[i].enqueueTask(kernels[i],nullptr,&events[i]); | ||
| queues[i].enqueueNDRangeKernel(kernels[i],cl::NullRange,cl::NDRange(1)); | ||
| for(int i=0;i<kernel_names.size();i++) | ||
| queues[i].finish(); | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -14,6 +14,7 @@ | |
|
|
||
| */ | ||
|
|
||
| #include "smi_generated_device.cl" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These include (and the ones in the successive file) should not be necessary with the old CMakeList |
||
|
|
||
| #pragma OPENCL EXTENSION cl_intel_channels : enable | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -14,6 +14,7 @@ | |
|
|
||
| */ | ||
|
|
||
| #include "smi_generated_device.cl" | ||
|
|
||
| #pragma OPENCL EXTENSION cl_intel_channels : enable | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this change necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new compiler version is not able to compile two cl files together, so we switched to compiling only the kernel file and then include the smi_generated_device.cl from there.