> -----Original Message-----
> From: Beignet [mailto:[email protected]] On Behalf Of
> Xiuli Pan
> Sent: Friday, September 9, 2016 1:41 PM
> To: [email protected]
> Cc: Li, Peng <[email protected]>; Pan, Xiuli <[email protected]>
> Subject: [Beignet] [PATCH] Backend: Refine replace src for write instruction
> 
> From: Pan Xiuli <[email protected]>
> 
> For write instruction src regs, they may be some header and need to be
> replaced with nomask mov.
Why should we set nomask for write operation? Could you give more details?

- Ruiling
> 
> Signed-off-by: Pan Xiuli <[email protected]>
> ---
>  backend/src/backend/gen_insn_selection.cpp | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/backend/src/backend/gen_insn_selection.cpp
> b/backend/src/backend/gen_insn_selection.cpp
> index 9a26cdc..d92c597 100644
> --- a/backend/src/backend/gen_insn_selection.cpp
> +++ b/backend/src/backend/gen_insn_selection.cpp
> @@ -1096,6 +1096,9 @@ namespace gbe
>          mov->state.flag = 0;
>          mov->state.subFlag = 0;
>        }
> +      // This reg may be header of send message.
> +      if (insn->isWrite())
> +        mov->state.noMask = 1;
>        if (this->isScalarReg(insn->src(regID).reg()))
>          mov->state.noMask = 1;
>        mov->dst(0) = gr;
> --
> 2.7.4
> 
> _______________________________________________
> Beignet mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/beignet
_______________________________________________
Beignet mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/beignet

Reply via email to