Yes, I include it in my patch, but it is good to make a separate patch for the fix. Looks good. Please merge it.
Thanks! Ruiling > -----Original Message----- > From: Beignet [mailto:[email protected]] On Behalf Of > Yang Rong > Sent: Monday, May 18, 2015 1:52 PM > To: [email protected] > Cc: Yang, Rong R > Subject: [Beignet] [PATCH] Fix a indirect register bug. > > Must init the fields indirect register used. > > Signed-off-by: Yang Rong <[email protected]> > --- > backend/src/backend/gen_register.hpp | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/backend/src/backend/gen_register.hpp > b/backend/src/backend/gen_register.hpp > index 80e143e..794498e 100644 > --- a/backend/src/backend/gen_register.hpp > +++ b/backend/src/backend/gen_register.hpp > @@ -837,8 +837,9 @@ namespace gbe > reg.file = GEN_GENERAL_REGISTER_FILE; > reg.address_mode = > GEN_ADDRESS_REGISTER_INDIRECT_REGISTER; > reg.width = width; > - reg.subnr = subnr; > + reg.a0_subnr = subnr; > reg.nr = 0; > + reg.addr_imm = 0; > reg.negation = 0; > reg.absolute = 0; > reg.vstride = vstride; > -- > 1.8.3.2 > > _______________________________________________ > Beignet mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/beignet
