This is an archive of the discontinued LLVM Phabricator instance.

[Clang][CUDA] Add descriptors, mappings, and features for missing CUDA and PTX versions
ClosedPublic

Authored by steffenlarsen on Jul 29 2021, 3:47 AM.

Details

Reviewers
tra
Summary

Adds missing descriptors and mappings for CUDA 11.4 and older. Also adds missing sub-target features for PTX 7.4 and older.

Authored-by: Steffen Larsen <steffen.larsen@codeplay.com>

Diff Detail

Event Timeline

steffenlarsen created this revision.Jul 29 2021, 3:47 AM
steffenlarsen requested review of this revision.Jul 29 2021, 3:47 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 29 2021, 3:47 AM
tra added inline comments.Jul 29 2021, 1:50 PM
clang/include/clang/Basic/BuiltinsNVPTX.def
821–823

You need to pop new macros here.

Made sure the PTX73 and PTX74 macros are popped correctly.

steffenlarsen marked an inline comment as done.Jul 30 2021, 1:20 AM
steffenlarsen added inline comments.
clang/include/clang/Basic/BuiltinsNVPTX.def
821–823

My bad! It has been corrected.

bader added a subscriber: bader.Nov 18 2021, 1:42 PM

@tra, ping.
@steffenlarsen, does it make sense to add support for recently released 11.5 as well?

tra accepted this revision.Nov 18 2021, 2:10 PM

I think this patch has been obsoleted by https://reviews.llvm.org/D113249 which has already landed.

My apologies for letting the patch slip through the cracks.

This revision is now accepted and ready to land.Nov 18 2021, 2:10 PM
steffenlarsen closed this revision.Nov 18 2021, 2:52 PM
steffenlarsen marked an inline comment as done.

I think this patch has been obsoleted by https://reviews.llvm.org/D113249 which has already landed.

My apologies for letting the patch slip through the cracks.

No harm, no foul! Thank you for pointing to the new patch. I agree, it seems that these changes are obsolete so I'll close this. :)