[Beignet] [PATCH 1/5] BDW: Forgot to set UIP of else in BDW.

Zhigang Gong zhigang.gong at linux.intel.com
Fri Oct 10 01:43:32 PDT 2014


This patchset LGTM, will push latter. Thanks.

On Thu, Oct 09, 2014 at 02:07:52PM +0800, Yang Rong wrote:
> GEN_OPCODE_ELSE also need set UIP, add it.
> 
> Signed-off-by: Yang Rong <rong.r.yang at intel.com>
> ---
>  backend/src/backend/gen8_encoder.cpp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/backend/src/backend/gen8_encoder.cpp b/backend/src/backend/gen8_encoder.cpp
> index f50ce47..6cb6e70 100644
> --- a/backend/src/backend/gen8_encoder.cpp
> +++ b/backend/src/backend/gen8_encoder.cpp
> @@ -273,7 +273,8 @@ namespace gbe
>      if(insn.header.opcode == GEN_OPCODE_ELSE)
>        uip = jip;
>  
> -    if (insn.header.opcode == GEN_OPCODE_IF) {
> +    if (insn.header.opcode == GEN_OPCODE_IF ||
> +        insn.header.opcode == GEN_OPCODE_ELSE) {
>        Gen8NativeInstruction *gen8_insn = &insn.gen8_insn;
>        this->setSrc0(&insn, GenRegister::immud(0));
>        gen8_insn->bits2.gen8_branch.uip = uip*8;
> -- 
> 1.9.1
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list