From 90b8666ff2f73e691ff34ffa87962a0922950b7d Mon Sep 17 00:00:00 2001 From: Karol Herbst Date: Wed, 7 Jun 2023 23:14:18 +0200 Subject: [PATCH] clc: relax spec constant validation Multiple values can have multiple spec constants assigned and vtn handles this just fine. So just drop that assert as we need it to run SyCL kernels. Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/9037 Fixes: a699844ffba ("microsoft/clc: Parse SPIR-V specialization consts into metadata") Signed-off-by: Karol Herbst Acked-by: Jesse Natalie Part-of: --- src/compiler/clc/clc_helpers.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/src/compiler/clc/clc_helpers.cpp b/src/compiler/clc/clc_helpers.cpp index 37d65e8c9b2..17400b07e36 100644 --- a/src/compiler/clc/clc_helpers.cpp +++ b/src/compiler/clc/clc_helpers.cpp @@ -338,7 +338,6 @@ public: uint32_t spec_id = ins->words[ins->operands[2].offset]; for (auto &c : specConstants) { if (c.second.id == spec_id) { - assert(c.first == id); return; } }