> -----Original Message----- > From: Beignet [mailto:[email protected]] On Behalf Of > Giuseppe Bilotta > Sent: Thursday, November 19, 2015 3:52 AM > To: [email protected] > Cc: Giuseppe Bilotta <[email protected]> > 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 > [email protected] > http://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/beignet
