Mesa (master): clover/nir: Check the result of spirv_to_nir

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 38bbfd3a57d68abdc88a93b436eac9f30a397b0f
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=38bbfd3a57d68abdc88a93b436eac9f30a397b0f

Author: Pierre Moreau <dev at pmoreau.org>
Date:   Tue May  5 13:13:19 2020 +0200

clover/nir: Check the result of spirv_to_nir

Fixes: deb04adf2ae ("clover: add support for passing kernels as nir to the driver")
Signed-off-by: Pierre Moreau <dev at pmoreau.org>
Reviewed-by: Karol Herbst <kherbst at redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4901>

---

 src/gallium/state_trackers/clover/nir/invocation.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/gallium/state_trackers/clover/nir/invocation.cpp b/src/gallium/state_trackers/clover/nir/invocation.cpp
index 87a98046cc1..46440d96e09 100644
--- a/src/gallium/state_trackers/clover/nir/invocation.cpp
+++ b/src/gallium/state_trackers/clover/nir/invocation.cpp
@@ -90,6 +90,11 @@ module clover::nir::spirv_to_nir(const module &mod, const device &dev,
       nir_shader *nir = spirv_to_nir(data, num_words, nullptr, 0,
                                      MESA_SHADER_KERNEL, name,
                                      &spirv_options, compiler_options);
+      if (!nir) {
+         r_log += "Translation from SPIR-V to NIR for kernel \"" + sym.name +
+                  "\" failed.\n";
+         throw build_error();
+      }
 
       nir->info.cs.local_size_variable = true;
       nir_validate_shader(nir, "clover");