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>
pgpke5a5pBpVc.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev