Hi!,

On vie, 2015-03-13 at 10:42 -0700, Matt Turner wrote:
> On Fri, Mar 13, 2015 at 3:41 AM, Antia Puentes <apuen...@igalia.com> wrote:
> > IvyBridge and Haswell PRM say that the JIP should be emitted
> > with type W but we were using UD. The previous implementation
> > did not show adverse effects, however changing the type to
> > D caused a GPU hang, see bug 84557; IMHO it is safer to
> > follow the specification thoroughly.
> > ---
> 
> The change seems fine to me, though changing the type to D never
> seemed like the thing that was actually causing the hang. It seemed
> much more likely that allowing the ELSE and ENDIF instructions to be
> compacted violated some assumption in our code and we misaligned some
> jump target by 8-bytes.
> 
> I should investigate more.
> 
> I'd probably remove the "however changing the type to D caused a GPU
> hang, see bug 84557" comment from the commit.

I will remove the comment.

> Reviewed-by: Matt Turner <matts...@gmail.com>

Thanks for reviewing!


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

Reply via email to