[Beignet] [PATCH 4/8] OCL20: handle device enqueue helper functions in the backend.

Song, Ruiling ruiling.song at intel.com
Fri May 27 06:23:59 UTC 2016



> diff --git a/backend/src/llvm/llvm_gen_backend.cpp
> b/backend/src/llvm/llvm_gen_backend.cpp
> index 1c41207..033e7de 100644
> --- a/backend/src/llvm/llvm_gen_backend.cpp
> +++ b/backend/src/llvm/llvm_gen_backend.cpp
> @@ -1474,9 +1474,6 @@ namespace gbe
>        return;
>      }
>      if (isa<GlobalVariable>(c)) {
> -      const GlobalVariable *GV = cast<GlobalVariable>(c);
> -
> -      unsigned valueAddrSpace = GV->getType()->getAddressSpace();
>        ir::Constant cc = unit.getConstantSet().getConstant(c->getName());
>        unsigned int defOffset = cc.getOffset();
> 
> @@ -1582,10 +1579,16 @@ namespace gbe
>            offset += sizeof(short);
>            break;
>          }
> -      default: {
> -        c->dump();
> -        NOT_IMPLEMENTED;
> -               }
> +      case Type::TypeID::PointerTyID:
> +        {
> +          //c->dump();
> 
> Need to be removed.
The pointer type in program scope variable is already handled by the above ConstantExpr condition check. Suppose you don't need this anymore.

> 
> +          break;
> +        }
> +      default:
> +        {
> +          c->dump();
> +          NOT_IMPLEMENTED;
> +        }
>      }
>    }



More information about the Beignet mailing list