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

glm::reflect definition doesn't have GLM_FUNC_QUALIFIER, so doesnt work on CUDA #43

Closed
Verdagon opened this issue Feb 2, 2013 · 2 comments
Closed

glm::reflect definition doesn't have GLM_FUNC_QUALIFIER, so doesnt work on CUDA #43

Verdagon opened this issue Feb 2, 2013 · 2 comments
Assignees
@Groovounet
Labels
bug
Milestone
GLM 0.9.4

Comments

@Verdagon
Copy link

Verdagon commented Feb 2, 2013

We tried to use glm::reflect in our CUDA kernel, but it complained that we were trying to call it from a device.

Putting GLM_FUNC_QUALIFIER in front of it (func_geometric.inl:274), like the rest of the functions, made it work.
@Verdagon
Copy link
Author

Similar issues in a lot of other places, too. Adding GLM_FUNC_DECL in front of type_half.hpp:39-40 fixes some cuda-related warnings, for example.

@ghost ghost assigned Groovounet Feb 13, 2013
Groovounet pushed a commit that referenced this issue Feb 14, 2013
Fixed CUDA warnings and errors, #43
0df4b94
@Groovounet
Copy link
Member

Hi,

I fixed the issues you have reported in GLM 0.9.4 branch. If you find others, please let me know.

Thanks for contributing!
Christophe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Groovounet Groovounet

Labels
bug
Projects
None yet
Milestone
GLM 0.9.4
Development

No branches or pull requests
2 participants
@Verdagon @Groovounet