Re: [Mesa-dev] [PATCH] i965/clip: Fix brw_clip_unfilled.c/compute_offset's assembly.

2014-08-06 Thread Chris Forbes
Reviewed-by: Chris Forbes On Wed, Aug 6, 2014 at 6:57 PM, Kenneth Graunke wrote: > Due to the destination register width of 1 or 2, these instructions get > ExecSize 1 or 2. But dir and offset (used as src0) are both registers > of width 4, violating the execsize >= width assertion. > > I hones

[Mesa-dev] [PATCH] i965/clip: Fix brw_clip_unfilled.c/compute_offset's assembly.

2014-08-05 Thread Kenneth Graunke
Due to the destination register width of 1 or 2, these instructions get ExecSize 1 or 2. But dir and offset (used as src0) are both registers of width 4, violating the execsize >= width assertion. I honestly don't think this could have ever worked. Fixes Piglit's polygon-offset and polygon-mode-