Fix error return and TEST_PASS for negative SVM tests (KhronosGroup#2027)

niranjanjoshi121 · web-flow · commit a406b340913f · 2024-08-06T22:50:21.000+05:30
Fix error return and TEST_PASS for negative SVM tests Fix review comments from KhronosGroup#1802 Fixes KhronosGroup#1964
diff --git a/test_conformance/SVM/test_allocate_shared_buffer_negative.cpp b/test_conformance/SVM/test_allocate_shared_buffer_negative.cpp
@@ -60,7 +60,7 @@ int test_svm_allocate_shared_buffer_negative(cl_device_id deviceID,
     // under construction...
     err = create_cl_objects(deviceID, NULL, &context, &program, &queues[0],
                             &num_devices, CL_DEVICE_SVM_COARSE_GRAIN_BUFFER);
-    if (err) return -1;
+    if (err) return err;
 
     size_t size = 1024;
 
@@ -98,5 +98,5 @@ int test_svm_allocate_shared_buffer_negative(cl_device_id deviceID,
         clSVMFree(context, pBufData1);
     }
 
-    return 0;
+    return TEST_PASS;
 }

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ int test_svm_allocate_shared_buffer_negative(cl_device_id deviceID,`
`60`	`60`	`// under construction...`
`61`	`61`	`err = create_cl_objects(deviceID, NULL, &context, &program, &queues[0],`
`62`	`62`	`&num_devices, CL_DEVICE_SVM_COARSE_GRAIN_BUFFER);`
`63`		`- if (err) return -1;`
	`63`	`+ if (err) return err;`
`64`	`64`
`65`	`65`	`size_t size = 1024;`
`66`	`66`
`@@ -98,5 +98,5 @@ int test_svm_allocate_shared_buffer_negative(cl_device_id deviceID,`
`98`	`98`	`clSVMFree(context, pBufData1);`
`99`	`99`	`}`
`100`	`100`
`101`		`- return 0;`
	`101`	`+ return TEST_PASS;`
`102`	`102`	`}`