From 402fbcca936b3ecc8ede079f5cb1f87c818811b4 Mon Sep 17 00:00:00 2001 From: Yan Date: Wed, 23 Oct 2019 16:34:42 +0000 Subject: [PATCH] Enable UpdateDynamicOffsetsMultipleTimesComputePipeline case Austin added barriers for storage buffer in compute pipeline in this patch (https://dawn-review.googlesource.com/c/dawn/+/12301). This case now can work fine. Bug: dawn:236 Change-Id: Id449ae5053f1f018ea95c271bff7e4ab2180a937 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/12640 Commit-Queue: Austin Eng Reviewed-by: Corentin Wallez Reviewed-by: Austin Eng --- src/tests/end2end/DynamicBufferOffsetTests.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/tests/end2end/DynamicBufferOffsetTests.cpp b/src/tests/end2end/DynamicBufferOffsetTests.cpp index ec78996fb1..b44ca6ef99 100644 --- a/src/tests/end2end/DynamicBufferOffsetTests.cpp +++ b/src/tests/end2end/DynamicBufferOffsetTests.cpp @@ -381,8 +381,7 @@ TEST_P(DynamicBufferOffsetTests, UpdateDynamicOffestsMultipleTimesRenderPipeline } // Setting multiple dynamic offsets for the same bindgroup in one compute pass. -// TODO(shaobo.yan@intel.com) : enable this test after resolving dawn issue 198. -TEST_P(DynamicBufferOffsetTests, DISABLED_UpdateDynamicOffestsMultipleTimesComputePipeline) { +TEST_P(DynamicBufferOffsetTests, UpdateDynamicOffsetsMultipleTimesComputePipeline) { dawn::ComputePipeline pipeline = CreateComputePipeline(); std::array offsets = {kMinDynamicBufferOffsetAlignment,