[Beignet] [PATCH] Fix sizing error for bitfield

[Song, Ruiling]

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Giuseppe Bilotta
> Sent: Thursday, November 19, 2015 3:52 AM
> To: beignet at lists.freedesktop.org
> Cc: Giuseppe Bilotta <giuseppe.bilotta at gmail.com>
> Subject: [Beignet] [PATCH] Fix sizing error for bitfield
> 
> The mergeable field was define as an uint32_t, but MAX_SRC_NUM is now
> 40, so we need at least an uint64_t.
> ---
>  backend/src/backend/gen_insn_selection.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/backend/src/backend/gen_insn_selection.cpp
> b/backend/src/backend/gen_insn_selection.cpp
> index 84cfb6d..e1cf6f7 100644
> --- a/backend/src/backend/gen_insn_selection.cpp
> +++ b/backend/src/backend/gen_insn_selection.cpp
> @@ -279,7 +279,7 @@ namespace gbe
>      /*! Instruction that needs to be matched */
>      const ir::Instruction &insn;
>      /*! When sources have been overwritten, a child insn cannot be merged
> */
> -    uint32_t mergeable:ir::Instruction::MAX_SRC_NUM;
> +    uint64_t mergeable:ir::Instruction::MAX_SRC_NUM;
>      /*! Number of children we have in the pattern */
>      uint32_t childNum:7;
>      /*! A root must be generated, no matter what */
> --
> 2.6.0.rc2.233.g6dd8a9a.dirty

Hi Giuseppe,

Thanks for the fix, and the patch LGTM.

Ruiling
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet