Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2018-02-05 Thread Iago Toral
On Sun, 2018-02-04 at 14:40 -0500, Connor Abbott wrote: > On Mon, Dec 11, 2017 at 11:01 AM, Jason Ekstrand t> wrote: > > On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral > > wrote: > > > > > > This didn't get any reviews yet. Any takers? > > > > > > On Fri,

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2018-02-05 Thread Iago Toral
On Sun, 2018-02-04 at 10:58 -0800, Matt Turner wrote: > On Wed, Dec 13, 2017 at 11:21 PM, Iago Toral > wrote: > > On Tue, 2017-12-12 at 08:20 +0100, Iago Toral wrote: > > > > On Mon, 2017-12-11 at 08:01 -0800, Jason Ekstrand wrote: > > > > On Mon, Dec 11, 2017 at 12:55 AM,

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2018-02-04 Thread Connor Abbott
On Mon, Dec 11, 2017 at 11:01 AM, Jason Ekstrand wrote: > On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral wrote: >> >> This didn't get any reviews yet. Any takers? >> >> On Fri, 2017-12-01 at 13:46 +0100, Iago Toral Quiroga wrote: >> > Otherwise loop

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2018-02-04 Thread Matt Turner
On Wed, Dec 13, 2017 at 11:21 PM, Iago Toral wrote: > On Tue, 2017-12-12 at 08:20 +0100, Iago Toral wrote: > > On Mon, 2017-12-11 at 08:01 -0800, Jason Ekstrand wrote: > > On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral wrote: > > This didn't get any reviews

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2017-12-13 Thread Iago Toral
On Tue, 2017-12-12 at 08:20 +0100, Iago Toral wrote: > On Mon, 2017-12-11 at 08:01 -0800, Jason Ekstrand wrote: > > On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral > > wrote: > > > This didn't get any reviews yet. Any takers? > > > > > > > > > > > > On Fri, 2017-12-01 at 13:46

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2017-12-11 Thread Iago Toral
On Mon, 2017-12-11 at 08:01 -0800, Jason Ekstrand wrote: > On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral > wrote: > > This didn't get any reviews yet. Any takers? > > > > > > > > On Fri, 2017-12-01 at 13:46 +0100, Iago Toral Quiroga wrote: > > > > > Otherwise loop unrolling

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2017-12-11 Thread Jason Ekstrand
On Mon, Dec 11, 2017 at 12:55 AM, Iago Toral wrote: > This didn't get any reviews yet. Any takers? > > On Fri, 2017-12-01 at 13:46 +0100, Iago Toral Quiroga wrote: > > Otherwise loop unrolling will fail to see the actual cost of > > the unrolling operations when the loop body

Re: [Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2017-12-11 Thread Iago Toral
This didn't get any reviews yet. Any takers? On Fri, 2017-12-01 at 13:46 +0100, Iago Toral Quiroga wrote: > Otherwise loop unrolling will fail to see the actual cost of > the unrolling operations when the loop body contains 64-bit integer > instructions, and very specially when the divmod64

[Mesa-dev] [PATCH] i965/nir: do int64 lowering before optimization

2017-12-01 Thread Iago Toral Quiroga
Otherwise loop unrolling will fail to see the actual cost of the unrolling operations when the loop body contains 64-bit integer instructions, and very specially when the divmod64 lowering applies, since its lowering is quite expensive. Without this change, some in-development CTS tests for int64