[Beignet] [PATCH] GBE: fix two mix pointer bug.

Song, Ruiling ruiling.song at intel.com
Mon Jan 9 04:15:40 UTC 2017


> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Yang Rong
> Sent: Friday, January 6, 2017 10:59 PM
> To: beignet at lists.freedesktop.org
> Cc: Yang, Rong R <rong.r.yang at 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.yang at 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


More information about the Beignet mailing list