Enable the BoundArrayAccessors tint transform Change-Id: I44cbfbeb09dadcf22ae77a85cef0aff3bc79b16b Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/42860 Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Ben Clayton <bclayton@google.com>
diff --git a/src/dawn_native/ShaderModule.cpp b/src/dawn_native/ShaderModule.cpp index 1ae9379..96b97d9 100644 --- a/src/dawn_native/ShaderModule.cpp +++ b/src/dawn_native/ShaderModule.cpp
@@ -1102,9 +1102,7 @@ transformManager.append(std::make_unique<tint::transform::EmitVertexPointSize>()); transformManager.append(std::make_unique<tint::transform::Spirv>()); if (GetDevice()->IsRobustnessEnabled()) { - // TODO(enga): Run the Tint BoundArrayAccessors transform instead of the SPIRV Tools - // one, but it appears to crash after running VertexPulling. - // transformManager.append(std::make_unique<tint::transform::BoundArrayAccessors>()); + transformManager.append(std::make_unique<tint::transform::BoundArrayAccessors>()); } tint::Program program; @@ -1117,9 +1115,6 @@ } std::vector<uint32_t> spirv = generator.result(); - if (GetDevice()->IsRobustnessEnabled()) { - DAWN_TRY_ASSIGN(spirv, RunRobustBufferAccessPass(spirv)); - } DAWN_TRY(ValidateSpirv(spirv.data(), spirv.size())); return std::move(spirv); }