fix #265 - spelling errors in comments and print statements (#276)

abhiShandy · Kent Knox · commit 80288680f069 · 2016-07-01T13:56:02.000-05:00
* fix #265 - spelling errors in comments and print statements * correct the argument order in doxygen comments
diff --git a/src/clBLAS.h b/src/clBLAS.h
@@ -7809,9 +7809,9 @@ clblasDtrmm(
  * @param[in] M         Number of rows in matrix \b B.
  * @param[in] N         Number of columns in matrix \b B.
  * @param[in] alpha     The factor of matrix \b A.
+ * @param[in] A         Buffer object storing matrix \b A.
  * @param[in] offA      Offset of the first element of the matrix \b A in the
  *                      buffer object. Counted in elements.
- * @param[in] A         Buffer object storing matrix \b A.
  * @param[in] lda       Leading dimension of matrix \b A. For detailed
  *                      description, see clblasStrmm().
  * @param[out] B        Buffer object storing matrix \b B.
diff --git a/src/include/kerngen.h b/src/include/kerngen.h
@@ -244,7 +244,7 @@ kgenSyncFormatting(
  *
  * @return 0 on success; -1 if the source code exceeds the buffer,
  *           or level of the code nesting is not zero, or the returned
- *           type is not defined, or there is not a paranthesis opening
+ *           type is not defined, or there is not a parenthesis opening
  *           the argument list
  */
 int
diff --git a/src/library/blas/generic/solution_seq_make.c b/src/library/blas/generic/solution_seq_make.c
@@ -214,7 +214,7 @@ selectVectorization(
         }
 
 		//
-		// Routines that dont use LDS have to be below the isLdsUsed() code
+		// Routines that don't use LDS have to be below the isLdsUsed() code
 		//
 		if (step->funcID == CLBLAS_GEMM2)
 		{
diff --git a/src/library/blas/gens/clTemplates/gemm.cl b/src/library/blas/gens/clTemplates/gemm.cl
@@ -98,7 +98,7 @@ __kernel void GEMM_NN__KERNEL ( __global %TYPE const * restrict _A, __global %TY
     // %V     - Vectoring Width
     // %PANEL(*) - Panel Width to access Rows of A and Columns of B
     //               Right now, %V is assumed to be the panel width.
-    //               We dont use %PANEL in the current implementation.
+    //               We don't use %PANEL in the current implementation.
     //
     MV = M;
     #ifndef TAIL_RUN
@@ -118,7 +118,7 @@ __kernel void GEMM_NN__KERNEL ( __global %TYPE const * restrict _A, __global %TY
         bidX = ( get_group_id(0) / ( blockDimY));
         //
         // Note:
-        // Using the new Map function does not yeild any performnce gain.
+        // Using the new Map function does not yield any performnce gain.
         // In fact, it degraded the performance
         // Keep this commented.
         //
@@ -613,7 +613,7 @@ __kernel void GEMM_NT__KERNEL ( __global %TYPE const * restrict _A, __global %TY
     // %V     - Vectoring Width
     // %PANEL(*) - Panel Width to access Rows of A and Columns of B
     //               Right now, %V is assumed to be the panel width.
-    //               We dont use %PANEL in the current implementation.
+    //               We don't use %PANEL in the current implementation.
     //
     MV = M;
     NV = N;
@@ -1141,7 +1141,7 @@ __kernel void GEMM_TN__KERNEL ( __global %TYPE const * restrict _A, __global %TY
     // %V     - Vectoring Width
     // %PANEL(*) - Panel Width to access Rows of A and Columns of B
     //               Right now, %V is assumed to be the panel width.
-    //               We dont use %PANEL in the current implementation.
+    //               We don't use %PANEL in the current implementation.
     //
     MV = M;
     #ifndef TAIL_RUN
diff --git a/src/library/blas/gens/clTemplates/symm.cl b/src/library/blas/gens/clTemplates/symm.cl
@@ -176,18 +176,18 @@ const char *SYMM_C_KERNEL= "
 			return SYMM_VECTOR_LOAD(A, M, lda, row, col);
 		}
 		#ifdef __SYMM_LOWER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_FIRST(A,M,K,lda,row,col)	%VLOAD(0, (&A[(col)*lda + (row)]))
 		#elif defined(__SYMM_UPPER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_FIRST(A,M,K,lda,row,col) 	%VLOADWITHINCXV2(0, (&A[(row)*lda + (col)]), lda)
 		#endif
 		#define LOADA_SECOND(A,M,K,lda,row,col)		SYMM_VECTOR_LOAD_USING_SCALAR(A, M, lda, row, col)
 		#ifdef __SYMM_LOWER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_THIRD(A,M,K,lda,row, col)	%VLOADWITHINCXV2(0, (&A[(row)*lda + (col)]), lda)
 		#elif defined(__SYMM_UPPER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_THIRD(A,M,K,lda,row, col)	%VLOAD(0, (&A[(col)*lda + (row)]))
 		#endif
 		#define LOADA_TAIL(A,M,K,lda,row,col) 		SYMM_VECTOR_LOAD_USING_SCALAR(A,M,lda,row,col)
@@ -217,18 +217,18 @@ const char *SYMM_C_KERNEL= "
 			return SYMM_VECTOR_LOAD(B, N, ldb, row, col);
 		}
 		#ifdef __SYMM_UPPER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_FIRST(B,K,N,ldb,row,col)	%VLOAD(0, (&B[(col)*(ldb) + (row)]))
 		#elif defined(__SYMM_LOWER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_FIRST(B,K,N,ldb,row,col)	%VLOADWITHINCXV2(0, (&B[(row)*(ldb)  + (col)]), ldb)
 		#endif
 		#define LOADB_SECOND(B,K,N,ldb,row,col)		SYMM_VECTOR_LOAD_USING_SCALAR(B, N, ldb, row, col)
 		#ifdef __SYMM_UPPER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_THIRD(B,K,N,ldb,row,col)	%VLOADWITHINCXV2(0, (&B[(row)*(ldb) + (col)]), ldb)
 		#elif defined(__SYMM_LOWER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_THIRD(B,K,N,ldb,row,col)	%VLOAD(0, (&B[(col)*(ldb) + (row)]))
 		#endif
 		#define LOADB_TAIL(B,K,N,ldb,row,col)		SYMM_VECTOR_LOAD_USING_SCALAR(B, N,ldb,row,col)
@@ -288,7 +288,7 @@ const char *SYMM_C_KERNEL= "
 		// %V 	- Vectoring Width
 		// %PANEL(*) - Panel Width to access Rows of A and Columns of B
 		//		   Right now, %V is assumed to be the panel width.
-		//		   We dont use %PANEL in the current implementation.
+		//		   We don't use %PANEL in the current implementation.
 		//
 		blockDimY = ((M-1) / (threadsY * %ITEMY)) + 1;
 		bidY = ( get_group_id(0) % ( blockDimY));
@@ -673,18 +673,18 @@ const char *SYMM_C_KERNEL_WORKING_EXCEPT_CSYMM_PROBLEM = "
 			return SYMM_VECTOR_LOAD(A, M, lda, row, col);
 		}
 		#ifdef __SYMM_LOWER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_FIRST(A,M,K,lda,row,col)	%VLOAD(0, (&A[(col)*lda + (row)]))
 		#elif defined(__SYMM_UPPER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_FIRST(A,M,K,lda,row,col) 	%VLOADWITHINCXV2(0, (&A[(row)*lda + (col)]), lda)
 		#endif
 		#define LOADA_SECOND(A,M,K,lda,row,col)		SYMM_VECTOR_LOAD_USING_SCALAR(A, M, lda, row, col)
 		#ifdef __SYMM_LOWER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_THIRD(A,M,K,lda,row, col)	%VLOADWITHINCXV2(0, (&A[(row)*lda + (col)]), lda)
 		#elif defined(__SYMM_UPPER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADA_THIRD(A,M,K,lda,row, col)	%VLOAD(0, (&A[(col)*lda + (row)]))
 		#endif
 		#define LOADA_TAIL(A,M,K,lda,row,col) 		SYMM_VECTOR_LOAD_USING_SCALAR(A,M,lda,row,col)
@@ -714,18 +714,18 @@ const char *SYMM_C_KERNEL_WORKING_EXCEPT_CSYMM_PROBLEM = "
 			return SYMM_VECTOR_LOAD(B, N, ldb, row, col);
 		}
 		#ifdef __SYMM_UPPER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_FIRST(B,K,N,ldb,row,col)	%VLOAD(0, (&B[(col)*(ldb) + (row)]))
 		#elif defined(__SYMM_LOWER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_FIRST(B,K,N,ldb,row,col)	%VLOADWITHINCXV2(0, (&B[(row)*(ldb)  + (col)]), ldb)
 		#endif
 		#define LOADB_SECOND(B,K,N,ldb,row,col)		SYMM_VECTOR_LOAD_USING_SCALAR(B, N, ldb, row, col)
 		#ifdef __SYMM_UPPER__
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_THIRD(B,K,N,ldb,row,col)	%VLOADWITHINCXV2(0, (&B[(row)*(ldb) + (col)]), ldb)
 		#elif defined(__SYMM_LOWER__)
-			// CHECK: KPRINTF Behaviour with so many parantheses - If fails, use parantheses in the caller
+			// CHECK: KPRINTF Behaviour with so many parentheses - If fails, use parentheses in the caller
 			#define LOADB_THIRD(B,K,N,ldb,row,col)	%VLOAD(0, (&B[(col)*(ldb) + (row)]))
 		#endif
 		#define LOADB_TAIL(B,K,N,ldb,row,col)		SYMM_VECTOR_LOAD_USING_SCALAR(B, N,ldb,row,col)
@@ -783,7 +783,7 @@ const char *SYMM_C_KERNEL_WORKING_EXCEPT_CSYMM_PROBLEM = "
 		// %V 	- Vectoring Width
 		// %PANEL(*) - Panel Width to access Rows of A and Columns of B
 		//		   Right now, %V is assumed to be the panel width.
-		//		   We dont use %PANEL in the current implementation.
+		//		   We don't use %PANEL in the current implementation.
 		//
 		blockDimY = ((M-1) / (threadsY * %ITEMY)) + 1;
 		bidY = ( get_group_id(0) % ( blockDimY));
diff --git a/src/library/blas/gens/clTemplates/trmv.cl b/src/library/blas/gens/clTemplates/trmv.cl
@@ -170,7 +170,7 @@ __kernel void %PREFIXtrmv_CU_kernel( __global %TYPE const* restrict _A, __global
 
 			//
 			// Only TARGET_WIDTH threads points are to be read from X-vector
-			// We dont't use VLOAD here because incx could be > 1
+			// We don't use VLOAD here because incx could be > 1
 			// Minimal prototyping shows that having separate loading code
 			// for incx value of 1 does not change anything in performance
 			// In fact, the extra IF costs us.
@@ -421,7 +421,7 @@ __kernel void %PREFIXtrmv_CL_kernel( __global %TYPE const* restrict _A, __global
 
 			//
 			// Only TARGET_WIDTH threads points are to be read from X-vector
-			// We dont't use VLOAD here because incx could be > 1
+			// We don't use VLOAD here because incx could be > 1
 			// Minimal prototyping shows that having separate loading code
 			// for incx value of 1 does not change anything in performance
 			// In fact, the extra IF costs us.
diff --git a/src/library/blas/gens/kprintf.cpp b/src/library/blas/gens/kprintf.cpp
@@ -854,7 +854,7 @@ void kprintf::handleComplexJoin(char **_src, char **_dst)
     {
         case SCALAR:
             //
-            // Dont do a thing...ComplexJoin not applicable for Real numbers
+            // Don't do a thing...ComplexJoin not applicable for Real numbers
             //
             break;
 
@@ -1547,7 +1547,7 @@ void kprintf::handleReduceSumReal(char **_src, char **_dst, int vlength)
     if (!vlength) //Can happen for SCALAR cases where source code contains this within COMPLEX define
     {
         //
-        // Dont generate a thing.
+        // Don't generate a thing.
         // The src pointer has already been advanced to next line
         // Just move on..
         //
diff --git a/src/library/blas/gens/trsv_gemv.cpp b/src/library/blas/gens/trsv_gemv.cpp
@@ -278,7 +278,7 @@ static bool isTransposeFeasible(size_t triangle, size_t blockSize, size_t vecLen
 /*
  * NOTE:
  * No-Transpose case - The code iterates along the X direction. Vectoring is along Y Direction.
- * Since we dont iterate on Y direction (triangle height), this fixes the "blocky" component of the blocksize.
+ * Since we don't iterate on Y direction (triangle height), this fixes the "blocky" component of the blocksize.
  * The blockSize then determines how much width the block has on X direction and thus the number of loops
  * can be calculated from that information.
  */
diff --git a/src/library/blas/include/kprintf.hpp b/src/library/blas/include/kprintf.hpp
@@ -77,7 +77,7 @@ class kprintf {
     void registerType(const char *baseType, int vecWidth, int internalVecWidth=1);
     void registerReducedTypes( const char* in, int div);
     void registerSuperTypes( const char* in, int mul);
-    char* mystrtok( char* in, const char* tok); //NOTE: strtok overwrites the string. we dont like that...
+    char* mystrtok( char* in, const char* tok); //NOTE: strtok overwrites the string. we don't like that...
     //
     // VLOAD %TYPE%V from (%PTYPE*) kind of memory locations
     // The Kernel writers should use "%TYPE" and "%TYPE%V" for kernel aguments, local variables etc..
diff --git a/src/library/blas/xtrsv.c b/src/library/blas/xtrsv.c
@@ -60,7 +60,7 @@ orchestrateNonTransposeTRSV(CLBlasKargs *kargs, ListHead *trtriSeq, ListHead *ge
 
 	if ((trtri->subdims->y) != (gemv->subdims->y))
 	{
-		printf("TRSV: WARNING:	TRTRI and GEMV dont have identical sub-divisions!!! %lu and %lu\n", trtri->subdims->y, gemv->subdims->y);
+		printf("TRSV: WARNING:	TRTRI and GEMV don't have identical sub-divisions!!! %lu and %lu\n", trtri->subdims->y, gemv->subdims->y);
 		return clblasNotImplemented;
 	} else {
 		#ifdef DEBUG_TRSV
@@ -166,7 +166,7 @@ orchestrateTransposeTRSV(CLBlasKargs *kargs, ListHead *trtriSeq, ListHead *gemvS
 
 	if ((trtri->subdims->y) != (gemv->subdims->y))
 	{
-		printf("TRSV: Transpose: WARNING: TRTRI and GEMV dont have identical sub-divisions!!! %lu and %lu\n", trtri->subdims->y, gemv->subdims->y);
+		printf("TRSV: Transpose: WARNING: TRTRI and GEMV don't have identical sub-divisions!!! %lu and %lu\n", trtri->subdims->y, gemv->subdims->y);
 		return clblasNotImplemented;
 	} else {
 		#ifdef DEBUG_TRSV
diff --git a/src/library/common/kerngen_core.c b/src/library/common/kerngen_core.c
@@ -110,7 +110,7 @@ searchFuncName(const char *source, size_t *len)
     char *name = NULL;
 
     /*
-     * Search the opening paranthesis. The word before it is
+     * Search the opening parenthesis. The word before it is
      * the function name
      */
     sep = strchr(source, '(');

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ kgenSyncFormatting(`
`244`	`244`	`*`
`245`	`245`	`* @return 0 on success; -1 if the source code exceeds the buffer,`
`246`	`246`	`* or level of the code nesting is not zero, or the returned`
`247`		`- * type is not defined, or there is not a paranthesis opening`
	`247`	`+ * type is not defined, or there is not a parenthesis opening`
`248`	`248`	`* the argument list`
`249`	`249`	`*/`
`250`	`250`	`int`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ selectVectorization(`
`214`	`214`	`}`
`215`	`215`
`216`	`216`	`//`
`217`		`- // Routines that dont use LDS have to be below the isLdsUsed() code`
	`217`	`+ // Routines that don't use LDS have to be below the isLdsUsed() code`
`218`	`218`	`//`
`219`	`219`	`if (step->funcID == CLBLAS_GEMM2)`
`220`	`220`	`{`