On Thu, 22 Dec 2011 10:18:05 +0800, Yuanhan Liu <yuanhan....@linux.intel.com> 
wrote:
> On Wed, Dec 21, 2011 at 05:57:35AM -0800, Eric Anholt wrote:
> > On Wed, 21 Dec 2011 17:33:41 +0800, Yuanhan Liu 
> > <yuanhan....@linux.intel.com> wrote:
> > > If dynamic instruction store size is enabled, while after the brw_JMPI()
> > > and before the brw_land_fwd_jump() function, the eu instruction store
> > > base address(p->store) may change. Thus, the safe way to reference the
> > > jmp instruction is by index instead of by the instruction address.
> > 
> > Our other instructions return the instruction pointer, I don't think
> > jmpi should be special in that respect.
> 
> Right. Fixed and how about the following patch?

Reviewed-by: Eric Anholt <e...@anholt.net>

Attachment: pgpke5a5pBpVc.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to