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: a699844ffb ("microsoft/clc: Parse SPIR-V specialization consts into metadata")
Signed-off-by: Karol Herbst <git@karolherbst.de>
Acked-by: Jesse Natalie <jenatali@microsoft.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23512>
(cherry picked from commit 90b8666ff2)
This commit is contained in:
Karol Herbst 2023-06-07 23:14:18 +02:00 committed by Eric Engestrom
parent c2f0fd28ac
commit f4f6d2e7cf
2 changed files with 1 additions and 2 deletions

View file

@ -3271,7 +3271,7 @@
"description": "clc: relax spec constant validation",
"nominated": true,
"nomination_type": 1,
"resolution": 0,
"resolution": 1,
"main_sha": null,
"because_sha": "a699844ffba20fb0bfe7103eb70938bac604748e"
},

View file

@ -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;
}
}