[PATCH] D56274: [OPENMP][NVPTX]Fix incompatibility of __syncthreads with LLVM, NFC.
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 28 13:33:29 PDT 2019
hfinkel added a comment.
> I reported several problems to NVIDIA.
Unfortunately, reporting problems to NVIDIA doesn't necessarily cause things to be fixed upstream. If you can (also) report problems occurring upstream to the upstream bug tracker, that will be greatly helpful.
The scheme I'd like to see us follow upstream is, when workaround are required, to leave the proper code in addition to the workarounds. Guard the workarounds with ifdefs so that we can use them for older versions as necessary, but use the proper version when compiling with newer compilers (once the bugs have been fixed). Over time, we'll be able to eliminate workarounds for compilers we no longer support.
CHANGES SINCE LAST ACTION
More information about the llvm-commits