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.

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

Memory leak in occaOpenCL.cpp #40

Open
jkozdon opened this issue Apr 22, 2015 · 3 comments
Open

Memory leak in occaOpenCL.cpp #40

jkozdon opened this issue Apr 22, 2015 · 3 comments

Comments

@jkozdon
Copy link

jkozdon commented Apr 22, 2015

I think that free template should have a delete command for the data
delete (OpenCLKernelData_t*) this->data;

I am also getting some memory leaks in occaParser.cpp (from the new on line 3522) and occaParserNodes.cpp (from the new on line 216), but I am not sure where in my code these functions are getting called so I haven't been able to track down the leak fully.

@dmed256
Copy link
Collaborator

dmed256 commented Apr 23, 2015

Oops, seems like I didn't implement the free() in OpenCL or CUDA

I'll add that tonight

As for the parser, I'm fixing that in the [restructure] branch, but it might take a bit of time until it's memory-leak free (will have to work on it after graduation, so about 5 weeks). Sorry about that =/

@jkozdon
Copy link
Author

jkozdon commented Apr 23, 2015

No problem. Just starting to develop with Lucas and figure I should let you know about things as I find them.

On Apr 22, 2015 5:07 PM, David Medina [email protected] wrote:

Oops, seems like I didn't implement the free() in OpenCL or CUDA

I'll add that tonight

As for the parser, I'm fixing that in the [restructure] branch, but it might take a bit of time until it's memory-leak free (will have to work on it after graduation, so about 5 weeks). Sorry about that =/


Reply to this email directly or view it on GitHubhttps://github.com//issues/40#issuecomment-95371933.

@dmed256
Copy link
Collaborator

dmed256 commented Apr 23, 2015

Sounds great, thanks for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants