Kévin Petit

Results 151 comments of Kévin Petit

I've got the test passing, we're not laying out the structure but all structures in the **PhysicalStorageBuffer** storage class must be explicitly laid out. I've hacked clspv to test but...

Here's the clspv hack I used for reference: ```cpp diff --git a/lib/SPIRVProducerPass.cpp b/lib/SPIRVProducerPass.cpp index d39555e5..f1eb95bc 100644 --- a/lib/SPIRVProducerPass.cpp +++ b/lib/SPIRVProducerPass.cpp @@ -1918,9 +1918,9 @@ SPIRVID SPIRVProducerPassImpl::getSPIRVType(Type *Ty, bool needs_layout) {...

Landing clvk in Debian is definitely something that's been at the back of my mind but Debian moves slowly and whatever version we land there first will be very the...

I agree automating the build of distro packages would be useful, I've created https://github.com/kpet/clvk/issues/546. >I see you are already using github actions? Do the resulting binaries get stored somewhere for...

@Randrianasulu In case that was not clear from @rjodinchr's message, the error you're seeing is not a problem in clvk but clvk reporting that the Vulkan implementation on the device...

Reposting my initial reply here (was hoping Github would copy the whole conversation): >Some of these could probably be set by default. We could introduce a flag to select "dev"...

This is likely because the Vulkan headers are too old and missing some definitions. What version of the Vulkan headers does Debian 11 ship? As an alternative to using the...

Another idea: you could try to add a `#ifdef` around `VK_VENDOR_ID_POCL`, similarly to how we've done it for `VK_VENDOR_ID_MOBILEYE`. If that's enough to get clvk to build against the Vulkan...

Great! Glad to hear that. Do you want to open a PR with the change?

We should get SPIRV-Headers to provide strings for capabilities.