This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Dropped unnecessary define when compiling deviceRTLs for NVPTX
ClosedPublic

Authored by tianshilei1992 on Jan 14 2021, 10:07 AM.

Details

Summary

The comment said CUDA 9 header files use the nv_weak attribute which
clang is not yet prepared to handle. It's three years ago and now things have
changed. Based on my test, removing the definition doesn't have any problem on
my machine with CUDA 11.1 installed.

Diff Detail

Event Timeline

tianshilei1992 requested review of this revision.Jan 14 2021, 10:07 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 14 2021, 10:07 AM
jdoerfert accepted this revision.Jan 14 2021, 10:09 AM

Sure. But don't get too distracted here. All of the nvcc stuff will go away.

This revision is now accepted and ready to land.Jan 14 2021, 10:09 AM

Sure. But don't get too distracted here. All of the nvcc stuff will go away.

We still need some target specific code in any case. If we can eventually get rid of CUDA headers, then this could be useless. :-)

Sure. But don't get too distracted here. All of the nvcc stuff will go away.

We still need some target specific code in any case. If we can eventually get rid of CUDA headers, then this could be useless. :-)

We don't for the device runtime. That is the cool thing ;)