From eb8f71eecbc44e64cd7814f53b27b42c43dd660b Mon Sep 17 00:00:00 2001 From: Joel Liang Date: Wed, 27 Dec 2017 13:16:00 +0800 Subject: APPBROWSER-230, APPBROWSER-300: Rename GCScheduler enqueue to dispatch, sync to memory_barrier Also fix the synchronisation issues between different kernels. Change-Id: Ib59d83ae8d5cc8b0bdf13e6f4958edccdab91ca4 Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/114594 Reviewed-by: Anthony Barbier Tested-by: Jenkins --- src/runtime/GLES_COMPUTE/IGCSimpleFunction.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'src/runtime/GLES_COMPUTE/IGCSimpleFunction.cpp') diff --git a/src/runtime/GLES_COMPUTE/IGCSimpleFunction.cpp b/src/runtime/GLES_COMPUTE/IGCSimpleFunction.cpp index 19f178f445..f2926b0a3f 100644 --- a/src/runtime/GLES_COMPUTE/IGCSimpleFunction.cpp +++ b/src/runtime/GLES_COMPUTE/IGCSimpleFunction.cpp @@ -38,8 +38,7 @@ void IGCSimpleFunction::run() { ARM_COMPUTE_ERROR_ON_MSG(!_kernel, "The child class didn't set the GLES kernel or function isn't configured"); - // FIXME(APPBROWSER-300): We may need to rename "enqueue" to "dispatch" and "sync" to "memory_barrier". - GCScheduler::get().enqueue(_border_handler, false); - GCScheduler::get().sync(); - GCScheduler::get().enqueue(*_kernel); + GCScheduler::get().dispatch(_border_handler, false); + GCScheduler::get().memory_barrier(); + GCScheduler::get().dispatch(*_kernel); } -- cgit v1.2.1