Re: [Mesa-dev] [PATCH v5] i965/vec4: refactor brw_vec4_copy_propagation.

2015-09-22 Thread Alejandro Piñeiro
On 22/09/15 19:27, Jason Ekstrand wrote: > On Tue, Sep 22, 2015 at 9:17 AM, Alejandro Piñeiro > wrote: >> >> On 22/09/15 18:09, Jason Ekstrand wrote: >>> On Tue, Sep 22, 2015 at 8:06 AM, Alejandro Piñeiro >>> wrote: Now it is more similar to brw_fs_copy_propagation, with three clear

Re: [Mesa-dev] [PATCH v5] i965/vec4: refactor brw_vec4_copy_propagation.

2015-09-22 Thread Jason Ekstrand
On Tue, Sep 22, 2015 at 9:17 AM, Alejandro Piñeiro wrote: > > > On 22/09/15 18:09, Jason Ekstrand wrote: >> On Tue, Sep 22, 2015 at 8:06 AM, Alejandro Piñeiro >> wrote: >>> Now it is more similar to brw_fs_copy_propagation, with three >>> clear stages: >>> >>> 1) Build up the value we are propag

Re: [Mesa-dev] [PATCH v5] i965/vec4: refactor brw_vec4_copy_propagation.

2015-09-22 Thread Alejandro Piñeiro
On 22/09/15 18:09, Jason Ekstrand wrote: > On Tue, Sep 22, 2015 at 8:06 AM, Alejandro Piñeiro > wrote: >> Now it is more similar to brw_fs_copy_propagation, with three >> clear stages: >> >> 1) Build up the value we are propagating as if it were the source of a >> single MOV: >> 2) Check that w

Re: [Mesa-dev] [PATCH v5] i965/vec4: refactor brw_vec4_copy_propagation.

2015-09-22 Thread Jason Ekstrand
On Tue, Sep 22, 2015 at 8:06 AM, Alejandro Piñeiro wrote: > Now it is more similar to brw_fs_copy_propagation, with three > clear stages: > > 1) Build up the value we are propagating as if it were the source of a > single MOV: > 2) Check that we can propagate that value > 3) Build the final value

[Mesa-dev] [PATCH v5] i965/vec4: refactor brw_vec4_copy_propagation.

2015-09-22 Thread Alejandro Piñeiro
Now it is more similar to brw_fs_copy_propagation, with three clear stages: 1) Build up the value we are propagating as if it were the source of a single MOV: 2) Check that we can propagate that value 3) Build the final value Previously everything was somewhat messed up, making the implementation