Re: [Mesa-dev] [PATCH 08/20] anv/pipeline: Unify 3DSTATE_GS emission

2016-11-14 Thread Timothy Arceri
On Mon, 2016-11-14 at 08:46 -0800, Jason Ekstrand wrote: > > > On Sat, Nov 12, 2016 at 2:54 PM, Timothy Arceri bora.com> wrote: > > On Sat, 2016-11-12 at 13:34 -0800, Jason Ekstrand wrote: > > > > I have two questions and two suggestions below.  > > > > With the suggestions addressed and assum

Re: [Mesa-dev] [PATCH 08/20] anv/pipeline: Unify 3DSTATE_GS emission

2016-11-14 Thread Jason Ekstrand
On Sat, Nov 12, 2016 at 2:54 PM, Timothy Arceri < timothy.arc...@collabora.com> wrote: > On Sat, 2016-11-12 at 13:34 -0800, Jason Ekstrand wrote: > > I have two questions and two suggestions below. > > With the suggestions addressed and assuming the answer to both > questions is yes, Patch 7-8 are

Re: [Mesa-dev] [PATCH 08/20] anv/pipeline: Unify 3DSTATE_GS emission

2016-11-12 Thread Timothy Arceri
On Sat, 2016-11-12 at 13:34 -0800, Jason Ekstrand wrote: I have two questions and two suggestions below.  With the suggestions addressed and assuming the answer to both questions is yes, Patch 7-8 are: Reviewed-by: Timothy Arceri > --- >  src/intel/vulkan/gen7_pipeline.c  | 48 +---