Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-05-05 Thread Jonathan Gray
On Fri, Apr 22, 2016 at 11:19:04AM -0700, Chad Versace wrote: > On 04/22/2016 10:50 AM, Jason Ekstrand wrote: > > > > > > On Fri, Apr 22, 2016 at 10:15 AM, Jonathan Gray > > wrote: > > > > On Fri, Apr 22, 2016 at 05:31:29PM +0100, Emil Velikov wrote: > >> On 22 April 2016

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-22 Thread Jonathan Gray
On Fri, Apr 22, 2016 at 10:50:33AM -0700, Jason Ekstrand wrote: > On Fri, Apr 22, 2016 at 10:15 AM, Jonathan Gray wrote: > > > On Fri, Apr 22, 2016 at 05:31:29PM +0100, Emil Velikov wrote: > > > On 22 April 2016 at 16:08, Jonathan Gray wrote: > > > > It is worth noting that the isl code extensive

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-22 Thread Chad Versace
On 04/22/2016 10:50 AM, Jason Ekstrand wrote: > > > On Fri, Apr 22, 2016 at 10:15 AM, Jonathan Gray > wrote: > > On Fri, Apr 22, 2016 at 05:31:29PM +0100, Emil Velikov wrote: >> On 22 April 2016 at 16:08, Jonathan Gray > > wrote: >>> It is worth not

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-22 Thread Jason Ekstrand
On Fri, Apr 22, 2016 at 10:15 AM, Jonathan Gray wrote: > On Fri, Apr 22, 2016 at 05:31:29PM +0100, Emil Velikov wrote: > > On 22 April 2016 at 16:08, Jonathan Gray wrote: > > > It is worth noting that the isl code extensively requires designated > > > initialisers on anonymous structs. It isn't

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-22 Thread Jonathan Gray
On Fri, Apr 22, 2016 at 05:31:29PM +0100, Emil Velikov wrote: > On 22 April 2016 at 16:08, Jonathan Gray wrote: > > It is worth noting that the isl code extensively requires designated > > initialisers on anonymous structs. It isn't clear to me when gcc introduced > > support for this but it isn'

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-22 Thread Emil Velikov
On 22 April 2016 at 16:08, Jonathan Gray wrote: > It is worth noting that the isl code extensively requires designated > initialisers on anonymous structs. It isn't clear to me when gcc introduced > support for this but it isn't in 4.2. > I think it should work for GCC 4.2 with -fms-extensions. W

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-22 Thread Jonathan Gray
It is worth noting that the isl code extensively requires designated initialisers on anonymous structs. It isn't clear to me when gcc introduced support for this but it isn't in 4.2. Would you accept patches to remove them? ___ mesa-dev mailing list mes

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-19 Thread Emil Velikov
Strictly speaking, if the compiler is GL/DRI agnostic, one could move it outside of src/mesa/drivers/dri and into src/intel. Although I'm not sure how much of that is true. On 16 April 2016 at 20:45, Jason Ekstrand wrote: Please add a note in the commit message to warn about Chad's experience. I

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-19 Thread Emil Velikov
On 19 April 2016 at 20:47, Chad Versace wrote: > On Sat 16 Apr 2016, Jason Ekstrand wrote: >> --- >> configure.ac | 3 ++- >> src/Makefile.am | 9 +++-- >> src/intel/Makefile.am | 4 >> src/mesa/drivers/dri/i965/Makefile.am |

Re: [Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-19 Thread Chad Versace
On Sat 16 Apr 2016, Jason Ekstrand wrote: > --- > configure.ac | 3 ++- > src/Makefile.am | 9 +++-- > src/intel/Makefile.am | 4 > src/mesa/drivers/dri/i965/Makefile.am | 7 ++- > 4 files changed, 15 insertions(+), 8 dele

[Mesa-dev] [PATCH 01/13] i965: Add a dependency on libisl

2016-04-16 Thread Jason Ekstrand
--- configure.ac | 3 ++- src/Makefile.am | 9 +++-- src/intel/Makefile.am | 4 src/mesa/drivers/dri/i965/Makefile.am | 7 ++- 4 files changed, 15 insertions(+), 8 deletions(-) diff --git a/configure.ac b/configure.ac ind