Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-02-04 Thread Francisco Jerez
Iago Toral writes: > On Mon, 2019-02-04 at 08:50 +0100, Iago Toral wrote: >> On Fri, 2019-02-01 at 11:23 -0800, Francisco Jerez wrote: >> > Iago Toral writes: >> > >> > > On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: >> > > > Iago Toral writes: >> > > > >> > > > > On Thu,

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-02-04 Thread Iago Toral
On Mon, 2019-02-04 at 08:50 +0100, Iago Toral wrote: > On Fri, 2019-02-01 at 11:23 -0800, Francisco Jerez wrote: > > Iago Toral writes: > > > > > On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: > > > > Iago Toral writes: > > > > > > > > > On Thu, 2019-01-24 at 11:45 -0800, Francisco

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-02-03 Thread Iago Toral
On Fri, 2019-02-01 at 11:23 -0800, Francisco Jerez wrote: > Iago Toral writes: > > > On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: > > > Iago Toral writes: > > > > > > > On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: > > > > > Iago Toral writes: > > > > > > > > > > >

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-02-01 Thread Francisco Jerez
Iago Toral writes: > On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: >> Iago Toral writes: >> >> > On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: >> > > Iago Toral writes: >> > > >> > > > On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: >> > > > > Iago Toral

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-02-01 Thread Iago Toral
On Fri, 2019-02-01 at 12:34 +0100, Iago Toral wrote: > On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: > > Iago Toral writes: > > > > > On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: > > > > Iago Toral writes: > > > > > > > > > On Wed, 2019-01-23 at 06:03 -0800, Francisco

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-02-01 Thread Iago Toral
On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: > Iago Toral writes: > > > On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: > > > Iago Toral writes: > > > > > > > On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: > > > > > Iago Toral Quiroga writes: > > > > > > > >

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-28 Thread Iago Toral
On Fri, 2019-01-25 at 12:54 -0800, Francisco Jerez wrote: > Iago Toral writes: > > > On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: > > > Iago Toral writes: > > > > > > > On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: > > > > > Iago Toral Quiroga writes: (...) > > >

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-25 Thread Francisco Jerez
Iago Toral writes: > On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: >> Iago Toral writes: >> >> > On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: >> > > Iago Toral Quiroga writes: >> > > >> > > > Commit c84ec70b3a72 implemented execution type promotion to 32- >> > > >

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-25 Thread Iago Toral
On Thu, 2019-01-24 at 10:22 -0800, Matt Turner wrote: > On Wed, Jan 23, 2019 at 4:18 AM Iago Toral Quiroga > wrote: > > > > Commit c84ec70b3a72 implemented execution type promotion to 32-bit > > for > > conversions involving half-float registers, which empirical testing > > suggested > > was

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-25 Thread Iago Toral
On Thu, 2019-01-24 at 11:45 -0800, Francisco Jerez wrote: > Iago Toral writes: > > > On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: > > > Iago Toral Quiroga writes: > > > > > > > Commit c84ec70b3a72 implemented execution type promotion to 32- > > > > bit > > > > for > > > >

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-24 Thread Francisco Jerez
Iago Toral writes: > On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: >> Iago Toral Quiroga writes: >> >> > Commit c84ec70b3a72 implemented execution type promotion to 32-bit >> > for >> > conversions involving half-float registers, which empirical testing >> > suggested >> > was

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-24 Thread Matt Turner
On Wed, Jan 23, 2019 at 6:03 AM Francisco Jerez wrote: > > Iago Toral Quiroga writes: > > > Commit c84ec70b3a72 implemented execution type promotion to 32-bit for > > conversions involving half-float registers, which empirical testing > > suggested > > was required, but it did not incorporate

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-24 Thread Matt Turner
On Wed, Jan 23, 2019 at 4:18 AM Iago Toral Quiroga wrote: > > Commit c84ec70b3a72 implemented execution type promotion to 32-bit for > conversions involving half-float registers, which empirical testing suggested > was required, but it did not incorporate this change into the assembly >

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-24 Thread Iago Toral
On Wed, 2019-01-23 at 06:03 -0800, Francisco Jerez wrote: > Iago Toral Quiroga writes: > > > Commit c84ec70b3a72 implemented execution type promotion to 32-bit > > for > > conversions involving half-float registers, which empirical testing > > suggested > > was required, but it did not

Re: [Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-23 Thread Francisco Jerez
Iago Toral Quiroga writes: > Commit c84ec70b3a72 implemented execution type promotion to 32-bit for > conversions involving half-float registers, which empirical testing suggested > was required, but it did not incorporate this change into the assembly > validator > logic. This commits adds

[Mesa-dev] [PATCH] intel/compiler: update validator to account for half-float exec type promotion

2019-01-23 Thread Iago Toral Quiroga
Commit c84ec70b3a72 implemented execution type promotion to 32-bit for conversions involving half-float registers, which empirical testing suggested was required, but it did not incorporate this change into the assembly validator logic. This commits adds that, preventing validation errors like