`urPlatformGet` & `urDeviceGet` should return valid error code. about unified-runtime

[pbalcer]

I think we should just remove that sentence from the docs. This seems like an extraneous check the implementation would have to do for little benefit. If we are worried that users might inadvertently calculate NumEntries to be zero, we can add a check for this in loader's validation layer (that hopefully will someday exist :-)).

From a cursory glance, linux syscalls don't check things like this (e.g., in writev/readv).

[veselypeta]

I'm not sure. This check in implemented in PI here https://github.com/intel/llvm/blob/sycl/sycl/plugins/level_zero/pi_level_zero.cpp#L2527. It also aligns with OpenCL https://registry.khronos.org/OpenCL/sdk/1.0/docs/man/xhtml/clGetDeviceIDs.html. However, if you believe that this can be done at the loader level then that seems fine with me. I don't know enough about the loader to comment.

[kbenzie]

I think, in the short term at least, there's value to adding these.

Once we have a validation layer we can reconsider where those types of checks should reside.