[Parallel_libs-commits] [PATCH] D23941: [StreamExecutor] Fix KernelSpec Doxygen

Jason Henline via Parallel_libs-commits parallel_libs-commits at lists.llvm.org
Fri Aug 26 12:44:25 PDT 2016


jhen created this revision.
jhen added a reviewer: jprice.
jhen added a subscriber: parallel_libs-commits.

There was a typo where \endcode was spelled as \encode and it was
keeping the whole file document from rendering. I also added in some \c
annotations for inline code stuff to make it look nicer.

https://reviews.llvm.org/D23941

Files:
  streamexecutor/include/streamexecutor/KernelSpec.h

Index: streamexecutor/include/streamexecutor/KernelSpec.h
===================================================================
--- streamexecutor/include/streamexecutor/KernelSpec.h
+++ streamexecutor/include/streamexecutor/KernelSpec.h
@@ -31,9 +31,9 @@
 /// the user. A simplified example workflow which a compiler might follow in the
 /// case of a CUDA kernel that is compiled to CUDA fatbin code is as follows:
 ///
-/// 1. The user defines a kernel function called UserKernel.
+/// 1. The user defines a kernel function called \c UserKernel.
 /// 2. The compiler compiles the kernel code into CUDA fatbin data and embeds
-///    that data into the host code at address __UserKernelFatbinAddress.
+///    that data into the host code at address \c __UserKernelFatbinAddress.
 /// 3. The compiler adds code at the beginning of the host code to instantiate a
 ///    MultiKernelLoaderSpec:
 ///    \code
@@ -43,16 +43,16 @@
 ///    \endcode
 /// 4. The compiler then adds code to the host code to add the fatbin data to
 ///    the new MultiKernelLoaderSpec, and to associate that data with the kernel
-///    name "UserKernel":
+///    name \c "UserKernel":
 ///    \code
 ///    namespace compiler_cuda_namespace {
 ///      UserKernelLoaderSpec.addCUDAFatbinInMemory(
 ///        __UserKernelFatbinAddress, "UserKernel");
 ///    } // namespace compiler_cuda_namespace
-///    \encode
+///    \endcode
 /// 5. The host code, having known beforehand that the compiler would initialize
 ///    a MultiKernelLoaderSpec based on the name of the CUDA kernel, makes use
-///    of the symbol cudanamespace::UserKernelLoaderSpec without defining it.
+///    of the symbol \c cudanamespace::UserKernelLoaderSpec without defining it.
 ///
 /// In the example above, the MultiKernelLoaderSpec instance created by the
 /// compiler can be used by the host code to create StreamExecutor kernel


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23941.69422.patch
Type: text/x-patch
Size: 1887 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/parallel_libs-commits/attachments/20160826/659fed5d/attachment.bin>


More information about the Parallel_libs-commits mailing list