[fuzz] Remove redundant workgroup size check
This check was added in:
https://dawn-review.googlesource.com/c/dawn/+/235334
It was required to avoid the subsequent IR validation step failing and
then triggering a fuzzing issue. Now that this validation failure
causes a skip instead of a crash, we do not need the extra check in
the fuzzer.
Change-Id: I607a41d19fd533cda7bb1159ab10cb1a33fd7f79
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/244395
Reviewed-by: dan sinclair <dsinclair@chromium.org>
Commit-Queue: James Price <jrprice@google.com>
diff --git a/src/tint/cmd/fuzz/ir/fuzz.cc b/src/tint/cmd/fuzz/ir/fuzz.cc
index 810f003..02c6964 100644
--- a/src/tint/cmd/fuzz/ir/fuzz.cc
+++ b/src/tint/cmd/fuzz/ir/fuzz.cc
@@ -89,25 +89,6 @@
return;
}
- // Workgroup sizes < 1 are checked by Dawn and the Tint binary,
- // should be skipped by the fuzzer.
- for (auto func : ir->functions) {
- if (func->Stage() != tint::core::ir::Function::PipelineStage::kCompute) {
- continue;
- }
-
- auto wg = func->WorkgroupSize().value();
- for (auto value : wg) {
- auto* cnst = value->As<core::ir::Constant>();
- TINT_ASSERT(cnst);
-
- auto val = cnst->Value()->ValueAs<int32_t>();
- if (val < 1) {
- return;
- }
- }
- }
-
// Validate the IR against the fuzzer's preconditions before running.
// We don't consider validation failure here to be an issue, as it only signals that
// there is a bug somewhere in the components run above. Those components have their own
