> -----Original Message----- > From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of > Yang Rong > Sent: Friday, January 6, 2017 10:59 PM > To: beignet@lists.freedesktop.org > Cc: Yang, Rong R <rong.r.y...@intel.com> > Subject: [Beignet] [PATCH] GBE: fix two mix pointer bug. > > 1. When update pointerOrigMap, only non-select and non-phi insn need > update second[0], if update select or phi's second[0], will over write the > info. > 2. For legacy A64 mix pointer, tmp register is start from insn.extra.elem. > > Signed-off-by: Yang Rong <rong.r.y...@intel.com>
The code LGTM. But I think it will be better to split as two patches. And you need to change the commit message. This is not related to A64. - Ruiling _______________________________________________ Beignet mailing list Beignet@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/beignet
