[Beignet] [PATCH 1/2] Backend: Resize the selection instruction max dst num

Yang, Rong R rong.r.yang at intel.com
Wed Aug 31 08:41:03 UTC 2016


LGTM, thanks.

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Xiuli Pan
> Sent: Monday, August 22, 2016 14:03
> To: beignet at lists.freedesktop.org
> Cc: Pan, Xiuli <xiuli.pan at intel.com>
> Subject: [Beignet] [PATCH 1/2] Backend: Resize the selection instruction max
> dst num
> 
> From: Pan Xiuli <xiuli.pan at intel.com>
> 
> Resize max dst num to 17 for the image block read8 function.
> 
> Signed-off-by: Pan Xiuli <xiuli.pan at intel.com>
> ---
>  backend/src/backend/gen_insn_selection.hpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/backend/src/backend/gen_insn_selection.hpp
> b/backend/src/backend/gen_insn_selection.hpp
> index 5e28ec9..14ac05f 100644
> --- a/backend/src/backend/gen_insn_selection.hpp
> +++ b/backend/src/backend/gen_insn_selection.hpp
> @@ -94,8 +94,8 @@ namespace gbe
>      void setDBGInfo(DebugInfo in) { DBGInfo = in; }
>      /*! No more than 40 sources (40 sources are used by vme for payload
> passing and setting) */
>      enum { MAX_SRC_NUM = 40 };
> -    /*! No more than 16 destinations (15 used by I64DIV/I64REM) */
> -    enum { MAX_DST_NUM = 16 };
> +    /*! No more than 17 destinations (17 used by image block read8) */
> +    enum { MAX_DST_NUM = 17 };
>      /*! State of the instruction (extra fields neeed for the encoding) */
>      GenInstructionState state;
>      union {
> --
> 2.7.4
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list