Kévin Petit

This is a grey area. Whether what Halide does is allowed or forbidden is not specified. The problem with what you're suggesting is that there is no explicit way to...

Removing the `__attribute__((destructor))` sounds good but I don't think OpenCL implementations are required (or should be required) to perform this cleanup. I think it's better for Halide apps to be...

Thanks for the report! >found it also needs -long-vector option.. maybe a consideration for enabling by default? Yeah, I'll have a look and consider enabling it by default, it might...

Thanks for the log! I'll dive into it when I get a chance. BTW, I've enabled support for long vectors by default now (https://github.com/kpet/clvk/commit/66e64a57980b3de158dd9cb76660355f26d532c3).

Hi, thanks for the report. There are two issues that I can spot: - `Error: 8-bit storage is not supported for SSBOs`. That means the Vulkan implementation you're targeting doesn't...

Thanks for the feedback! This is a clspv limitation that is being lifted. It may be that the work done so far (disabled by default) is sufficient for clpeak. You...

Thanks, that's an interesting data point. You can try adding `-cl-fast-relaxed-math` and `-cl-denorms-are-zero` as well. The global memory bandwidth probably is a clvk limitation (https://github.com/kpet/clvk/issues/98) and I suspect you're measuring...

Supporting the whole extension is quite a bit of work but it may be that your application only uses a small subset that's already supported. You can try adding the...

Apologies, I'd forgotten about some recent changes to clvk and clspv, FP64 now needs to be enabled in clspv as well. Removing the line that disables FP64 in clspv (https://github.com/kpet/clvk/blob/master/src/program.cpp#L764)...

Hi @KodeMunkie and thanks for your interest in clvk. You're not the first one to try to get FAH going (see https://github.com/kpet/clvk/issues/297). The first thing I suggest you check is...