richardmembarth created this revision. richardmembarth added reviewers: Anastasia, aaron.ballman. Herald added a subscriber: cfe-commits.
The current pretty-printer emits OpenCL-style memory spaces specifiers: __device, __constant, and __shared. The correct CUDA memory space specifiers are: __device__, __constant__, and __shared__: https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#variable-memory-space-specifiers Repository: rC Clang https://reviews.llvm.org/D54258 Files: lib/AST/TypePrinter.cpp Index: lib/AST/TypePrinter.cpp =================================================================== --- lib/AST/TypePrinter.cpp +++ lib/AST/TypePrinter.cpp @@ -1738,17 +1738,19 @@ case LangAS::opencl_private: break; case LangAS::opencl_constant: - case LangAS::cuda_constant: OS << "__constant"; break; case LangAS::opencl_generic: OS << "__generic"; break; case LangAS::cuda_device: - OS << "__device"; + OS << "__device__"; + break; + case LangAS::cuda_constant: + OS << "__constant__"; break; case LangAS::cuda_shared: - OS << "__shared"; + OS << "__shared__"; break; default: OS << "__attribute__((address_space(";
Index: lib/AST/TypePrinter.cpp =================================================================== --- lib/AST/TypePrinter.cpp +++ lib/AST/TypePrinter.cpp @@ -1738,17 +1738,19 @@ case LangAS::opencl_private: break; case LangAS::opencl_constant: - case LangAS::cuda_constant: OS << "__constant"; break; case LangAS::opencl_generic: OS << "__generic"; break; case LangAS::cuda_device: - OS << "__device"; + OS << "__device__"; + break; + case LangAS::cuda_constant: + OS << "__constant__"; break; case LangAS::cuda_shared: - OS << "__shared"; + OS << "__shared__"; break; default: OS << "__attribute__((address_space(";
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits