Incoherent Documentation/Validation between zeModuleCreate and ze_module_desc_t about level-zero HOT 4 CLOSED

Agreed, thanks for pointing this out. The intention here is that desc->pConstants is optional and nullptr should be an acceptable input. I'm working with our spec team to get the documentation updated, and we will post a fix to the validation layer soon. In the meantime, if you disable validation layer (or modify the code locally), passing nullptr for pConstants should work

from level-zero.

Yes, for now I use an empty ze_module_constants_t.

As far as the second issue is concerned (the check on the enum value), I have found several other instances where the test should be strict inequality < rather than <=, so the logic used to generate those seems problematic.

from level-zero.

Agreed that is an issue in multiple places. Another issue is that some flag fields are validated incorrectly. We will have a fix for this in the near future.

from level-zero.

These specific issues seem to have been fixed, I'll be closing this one to de-clutter.

from level-zero.