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

Add test for NVPTX intrinsics #3993

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thewilsonator
Copy link
Contributor

cc @JohanEngelen

#2556

Whats the best way to do this for all targets that LDC supports?

@JohanEngelen
Copy link
Member

This test should include actual usage of an intrinsic (adding dcompute_ as prefix to the filename would be nice too).
The idea is not to test the existence of the intrinsc file, but its correctness (for example, the test should catch what Kai forgot to add orginally: "The generated file will have to be marked @compute(CompileFor.deviceOnly), with the corresponding import of ldc.dcompute.").

Please make it part of #2556 (or add those changes to this PR).

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

Successfully merging this pull request may close these issues.

None yet

2 participants