Fix status returned by clSetKernelArg.

- In case of mismatched access qualifiers it would return
  CL_INVALID_KERNEL_ARGS. This is not correct, CL_INVALID_ARG_VALUE
  should be returned instead.

Change-Id: I0dc929e22269b4e34e347502c6cc4067e67ba165
This commit is contained in:
kkasper
2018-01-17 09:55:39 +01:00
committed by sys_ocldev
parent 0066daf495
commit b08de2733e
2 changed files with 5 additions and 5 deletions

View File

@ -174,14 +174,14 @@ TEST_F(KernelImageArgTest, givenImageWithWriteOnlyAccessAndReadOnlyArgWhenCheckC
cl_mem memObj = img.get();
retVal = pKernel->checkCorrectImageAccessQualifier(0, sizeof(memObj), &memObj);
EXPECT_EQ(retVal, CL_INVALID_KERNEL_ARGS);
EXPECT_EQ(retVal, CL_INVALID_ARG_VALUE);
retVal = clSetKernelArg(
pKernel,
0,
sizeof(memObj),
&memObj);
EXPECT_EQ(retVal, CL_INVALID_KERNEL_ARGS);
EXPECT_EQ(retVal, CL_INVALID_ARG_VALUE);
retVal = clSetKernelArg(
pKernel,
@ -189,7 +189,7 @@ TEST_F(KernelImageArgTest, givenImageWithWriteOnlyAccessAndReadOnlyArgWhenCheckC
sizeof(memObj),
&memObj);
EXPECT_EQ(retVal, CL_INVALID_KERNEL_ARGS);
EXPECT_EQ(retVal, CL_INVALID_ARG_VALUE);
retVal = clSetKernelArg(
pKernel,
@ -213,7 +213,7 @@ TEST_F(KernelImageArgTest, givenImageWithReadOnlyAccessAndWriteOnlyArgWhenCheckC
cl_mem memObj = img.get();
retVal = pKernel->checkCorrectImageAccessQualifier(0, sizeof(memObj), &memObj);
EXPECT_EQ(retVal, CL_INVALID_KERNEL_ARGS);
EXPECT_EQ(retVal, CL_INVALID_ARG_VALUE);
Image *image = NULL;
memObj = image;
retVal = pKernel->checkCorrectImageAccessQualifier(0, sizeof(memObj), &memObj);