Change SYCL_PI_TESTS to SYCL_UR_TESTS in CI

[omarahmed1111]

SYCL_PI_TESTS was deprecated and changed to SYCL_UR_TESTS.

[omarahmed1111]

@intel/llvm-gatekeepers please merge, Thanks!

[AlexeySachkov]

I think that a better change would be to completely drop the option:

llvm/sycl/unittests/CMakeLists.txt

Line 28 in 21018ab

# TODO UR tests require real hardware and must be moved to sycl/test-e2e.

This comment is totally outdated, because the only UR unit-test we have is:

llvm/sycl/unittests/ur/UrUtility.cpp

Lines 18 to 24 in 21018ab

	TEST(UrUtilityTest, CheckUrCastOCLEventVector) {
	// Current special case for vectors of OpenCL vectors. This may change in the
	// future.
	std::vector<cl_event> EVec{(cl_event)0};
	ur_native_handle_t ENativeHandle = detail::ur::cast<ur_native_handle_t>(EVec);
	EXPECT_EQ(ENativeHandle, (ur_native_handle_t)EVec[0]);
	}

The option seems to be some kind of legacy and that single test we have should be enabled by default

[omarahmed1111]

I think that a better change would be to completely drop the option:

llvm/sycl/unittests/CMakeLists.txt

Line 28 in 21018ab

# TODO UR tests require real hardware and must be moved to sycl/test-e2e.

This comment is totally outdated, because the only UR unit-test we have is:

llvm/sycl/unittests/ur/UrUtility.cpp

Lines 18 to 24 in 21018ab

	TEST(UrUtilityTest, CheckUrCastOCLEventVector) {
	// Current special case for vectors of OpenCL vectors. This may change in the
	// future.
	std::vector<cl_event> EVec{(cl_event)0};
	ur_native_handle_t ENativeHandle = detail::ur::cast<ur_native_handle_t>(EVec);
	EXPECT_EQ(ENativeHandle, (ur_native_handle_t)EVec[0]);
	}

The option seems to be some kind of legacy and that single test we have should be enabled by default

That make sense, removed the option.

[omarahmed1111]

@intel/llvm-gatekeepers please merge, Thanks!