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

Support C++ AMP #60

Closed
ssylvan opened this issue Mar 12, 2013 · 3 comments
Closed

Support C++ AMP #60

ssylvan opened this issue Mar 12, 2013 · 3 comments

Comments

@ssylvan
Copy link

ssylvan commented Mar 12, 2013

Adding "restrict (amp,cpu)" to the end of most all functions should be enough. Put it in a macro that's defined to nothing on non-AMP platforms. Use VS2012 (express edition is free) to test.

This way you could seamlessly use it for GPGPU using AMP.

@ssylvan ssylvan closed this as completed Mar 12, 2013
@ssylvan ssylvan reopened this Mar 12, 2013
@Groovounet
Copy link
Member

It's actually a lot more complex than what you were expecting. C++ AMP doesn't support 16 bits integers (Used for half) or built-in std functions like "floor".

It sounds very unlikely that such port happen unless C++ AMP became used enough that the work required to support it happen.

Thanks,
Christophe

@ssylvan
Copy link
Author

ssylvan commented May 18, 2013

Well, you could just leave the restrict(amp) thing off of any functions that wouldn't compile with it. Note: I'm not suggesting you actually implement any algorithms using AMP, just that you make it possible for others to use GLM in their AMP algorithms. If they can't use half or floor until AMP supports it, then so be it.

@Groovounet
Copy link
Member

if you willing to provide a pull request for GLM 0.9.5 branch, I would be very happy to accept it.

Thanks,
Christophe

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

No branches or pull requests

2 participants