[UR] Add urPlatformGetBackendOption invalid value check

Author: veselypeta

include/ur_api.h

@@ -544,6 +544,8 @@ urPlatformCreateWithNativeHandle(
 ///     - ::UR_RESULT_ERROR_INVALID_NULL_POINTER
 ///         + `NULL == pFrontendOption`
 ///         + `NULL == ppAdapterOption`
+///     - ::UR_RESULT_ERROR_INVALID_VALUE
+///         + If `pFrontendOption` is not a valid frontend option.
 UR_APIEXPORT ur_result_t UR_APICALL
 urPlatformGetBackendOption(
     ur_platform_handle_t hPlatform, ///< [in] handle of the platform instance.

scripts/core/platform.yml

@@ -188,6 +188,9 @@ params:
     - type: const char**
       name: ppAdapterOption
       desc: "[out] returns the correct adapter specific option based on the frontend option."
+returns:
+    - $X_RESULT_ERROR_INVALID_VALUE:
+      - "If `pFrontendOption` is not a valid frontend option."
 --- #--------------------------------------------------------------------------
 type: function
 desc: "Retrieve string representation of the underlying adapter specific result

source/loader/ur_libapi.cpp

@@ -273,6 +273,8 @@ ur_result_t UR_APICALL urPlatformCreateWithNativeHandle(
 ///     - ::UR_RESULT_ERROR_INVALID_NULL_POINTER
 ///         + `NULL == pFrontendOption`
 ///         + `NULL == ppAdapterOption`
+///     - ::UR_RESULT_ERROR_INVALID_VALUE
+///         + If `pFrontendOption` is not a valid frontend option.
 ur_result_t UR_APICALL urPlatformGetBackendOption(
     ur_platform_handle_t hPlatform, ///< [in] handle of the platform instance.
     const char

source/ur_api.cpp

@@ -229,6 +229,8 @@ ur_result_t UR_APICALL urPlatformCreateWithNativeHandle(
 ///     - ::UR_RESULT_ERROR_INVALID_NULL_POINTER
 ///         + `NULL == pFrontendOption`
 ///         + `NULL == ppAdapterOption`
+///     - ::UR_RESULT_ERROR_INVALID_VALUE
+///         + If `pFrontendOption` is not a valid frontend option.
 ur_result_t UR_APICALL urPlatformGetBackendOption(
     ur_platform_handle_t hPlatform, ///< [in] handle of the platform instance.
     const char

test/conformance/platform/urPlatformGetBackendOption.cpp

@@ -39,3 +39,10 @@ TEST_F(urPlatfromGetBackendOptionTest, InvalidNullPointerAdapterOption) {
     ASSERT_EQ_RESULT(UR_RESULT_ERROR_INVALID_NULL_POINTER,
                      urPlatformGetBackendOption(platform, "-O0", nullptr));
 }
+
+TEST_F(urPlatfromGetBackendOptionTest, InvalidValueFrontendOption) {
+    const char *adapterOption = nullptr;
+    ASSERT_EQ_RESULT(
+        UR_RESULT_ERROR_INVALID_VALUE,
+        urPlatformGetBackendOption(platform, "-sycl-sucks", &adapterOption));
+}