[Beignet] [PATCH] Remove global barrier assert.

Zhigang Gong zhigang.gong at linux.intel.com
Thu Jun 13 20:49:50 PDT 2013


With this patch, you treat a global memory fence the same as a local memory fence.
I haven't check the bspec details, but I am doubt it. Need some time to investigate
the spec.

On Fri, Jun 14, 2013 at 11:01:09AM +0800, Yang Rong wrote:
> Per openCL spec, Global memory barrier is consistent across work-items in
> a single work group, which is match the bspec's barrier. So remove global
> barrier assert.
> 
> Signed-off-by: Yang Rong <rong.r.yang at intel.com>
> ---
>  backend/src/backend/gen_insn_selection.cpp |    4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/backend/src/backend/gen_insn_selection.cpp b/backend/src/backend/gen_insn_selection.cpp
> index 88f9e94..3a139ea 100644
> --- a/backend/src/backend/gen_insn_selection.cpp
> +++ b/backend/src/backend/gen_insn_selection.cpp
> @@ -1607,10 +1607,6 @@ namespace gbe
>      INLINE bool emitOne(Selection::Opaque &sel, const ir::SyncInstruction &insn) const
>      {
>        using namespace ir;
> -      const uint32_t params = insn.getParameters();
> -      GBE_ASSERTM(params == syncLocalBarrier,
> -                  "Only barrier(CLK_LOCAL_MEM_FENCE) is supported right now "
> -                  "for the synchronization primitives");
>        const ir::Register reg = sel.reg(FAMILY_DWORD);
>  
>        sel.push();
> -- 
> 1.7.10.4
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list