Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Kenneth Graunke
On Monday, September 21, 2015 02:27:00 PM Matt Turner wrote: [snip] > Yeah, the block immediately above handles conditions like this: > > if (inst->opcode != BRW_OPCODE_MOV || > (inst->dst.file != GRF && inst->dst.file != MRF) || > inst->predicate || > inst->src[0].file != GRF || >

Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Matt Turner
On Mon, Sep 21, 2015 at 2:16 PM, Jason Ekstrand wrote: > On Mon, Sep 21, 2015 at 2:04 PM, Matt Turner wrote: >> With NIR: >> >> instructions in affected programs: 111508 -> 109193 (-2.08%) >> helped:507 > > *sigh* Any idea who's emitting these MOV's? If it's

Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Matt Turner
On Mon, Sep 21, 2015 at 2:16 PM, Kenneth Graunke wrote: > On Monday, September 21, 2015 02:04:00 PM Matt Turner wrote: >> With NIR: >> >> instructions in affected programs: 111508 -> 109193 (-2.08%) >> helped:507 >> >> Without NIR: >> >> instructions in affected

Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Jason Ekstrand
On Mon, Sep 21, 2015 at 2:17 PM, Jason Ekstrand wrote: > On Mon, Sep 21, 2015 at 2:16 PM, Kenneth Graunke > wrote: >> On Monday, September 21, 2015 02:04:00 PM Matt Turner wrote: >>> With NIR: >>> >>> instructions in affected programs: 111508 -> 109193 (-2.08%) >>> helped:

Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Jason Ekstrand
On Mon, Sep 21, 2015 at 2:16 PM, Kenneth Graunke wrote: > On Monday, September 21, 2015 02:04:00 PM Matt Turner wrote: >> With NIR: >> >> instructions in affected programs: 111508 -> 109193 (-2.08%) >> helped:507 >> >> Without NIR: >> >> instructions in affected

Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Kenneth Graunke
On Monday, September 21, 2015 02:04:00 PM Matt Turner wrote: > With NIR: > > instructions in affected programs: 111508 -> 109193 (-2.08%) > helped:507 > > Without NIR: > > instructions in affected programs: 28763 -> 28474 (-1.00%) > helped:

Re: [Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Jason Ekstrand
On Mon, Sep 21, 2015 at 2:04 PM, Matt Turner wrote: > With NIR: > > instructions in affected programs: 111508 -> 109193 (-2.08%) > helped:507 *sigh* Any idea who's emitting these MOV's? If it's vec_to_movs, we should probably fix it one of these days. In any

[Mesa-dev] [PATCH] i965/vec4: Detect and delete useless MOVs.

2015-09-21 Thread Matt Turner
With NIR: instructions in affected programs: 111508 -> 109193 (-2.08%) helped:507 Without NIR: instructions in affected programs: 28763 -> 28474 (-1.00%) helped:186 --- src/mesa/drivers/dri/i965/brw_vec4.cpp | 21 ++