Closed japaric closed 8 years ago
Yeah, this looks a lot cleaner than the way it was. The intrinsics seemed out of place when put next to other, more generic intrinsics. This looks good to me. Merge whenever you are ready.
I am going to change #12 to follow this pattern.
Merge whenever you are ready.
:+1:
I am going to change #12 to follow this pattern.
:+1:
See commit message for details.
This PR implements NVPTX intrinsics as platform instrinsics as suggested by @eddyb, and also add tests for the NVPTX backend.
r? @TheAustinSeven Also, #12 could be updated to use platform intrinsics as well.