Update command buffer equivalent as well.

aarongreig · aarongreig · commit dcd3c186b231 · 2024-07-10T10:18:11.000+01:00
diff --git a/include/ur_api.h b/include/ur_api.h
@@ -8387,7 +8387,6 @@ urCommandBufferAppendUSMMemcpyExp(
 ///     - ::UR_RESULT_ERROR_INVALID_SIZE
 ///         + `patternSize == 0 || size == 0`
 ///         + `patternSize > size`
-///         + `(patternSize & (patternSize - 1)) != 0`
 ///         + `size % patternSize != 0`
 ///         + If `size` is higher than the allocation size of `ptr`
 ///     - ::UR_RESULT_ERROR_INVALID_MEM_OBJECT
diff --git a/scripts/core/exp-command-buffer.yml b/scripts/core/exp-command-buffer.yml
@@ -408,7 +408,6 @@ returns:
     - $X_RESULT_ERROR_INVALID_SIZE:
         - "`patternSize == 0 || size == 0`"
         - "`patternSize > size`"
-        - "`(patternSize & (patternSize - 1)) != 0`"
         - "`size % patternSize != 0`"
         - "If `size` is higher than the allocation size of `ptr`"
     - $X_RESULT_ERROR_INVALID_MEM_OBJECT
diff --git a/source/loader/layers/validation/ur_valddi.cpp b/source/loader/layers/validation/ur_valddi.cpp
@@ -8085,10 +8085,6 @@ __urdlllocal ur_result_t UR_APICALL urCommandBufferAppendUSMFillExp(
             return UR_RESULT_ERROR_INVALID_SIZE;
         }
 
-        if ((patternSize & (patternSize - 1)) != 0) {
-            return UR_RESULT_ERROR_INVALID_SIZE;
-        }
-
         if (size % patternSize != 0) {
             return UR_RESULT_ERROR_INVALID_SIZE;
         }
diff --git a/source/loader/ur_libapi.cpp b/source/loader/ur_libapi.cpp
@@ -7539,7 +7539,6 @@ ur_result_t UR_APICALL urCommandBufferAppendUSMMemcpyExp(
 ///     - ::UR_RESULT_ERROR_INVALID_SIZE
 ///         + `patternSize == 0 || size == 0`
 ///         + `patternSize > size`
-///         + `(patternSize & (patternSize - 1)) != 0`
 ///         + `size % patternSize != 0`
 ///         + If `size` is higher than the allocation size of `ptr`
 ///     - ::UR_RESULT_ERROR_INVALID_MEM_OBJECT
diff --git a/source/ur_api.cpp b/source/ur_api.cpp
@@ -6403,7 +6403,6 @@ ur_result_t UR_APICALL urCommandBufferAppendUSMMemcpyExp(
 ///     - ::UR_RESULT_ERROR_INVALID_SIZE
 ///         + `patternSize == 0 || size == 0`
 ///         + `patternSize > size`
-///         + `(patternSize & (patternSize - 1)) != 0`
 ///         + `size % patternSize != 0`
 ///         + If `size` is higher than the allocation size of `ptr`
 ///     - ::UR_RESULT_ERROR_INVALID_MEM_OBJECT

Original file line number	Diff line number	Diff line change
`@@ -8085,10 +8085,6 @@ __urdlllocal ur_result_t UR_APICALL urCommandBufferAppendUSMFillExp(`
`8085`	`8085`	`return UR_RESULT_ERROR_INVALID_SIZE;`
`8086`	`8086`	`}`
`8087`	`8087`
`8088`		`- if ((patternSize & (patternSize - 1)) != 0) {`
`8089`		`- return UR_RESULT_ERROR_INVALID_SIZE;`
`8090`		`- }`
`8091`		`-`
`8092`	`8088`	`if (size % patternSize != 0) {`
`8093`	`8089`	`return UR_RESULT_ERROR_INVALID_SIZE;`
`8094`	`8090`	`}`