Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 12:04 PM, Ilia Mirkin wrote: > On Wed, Dec 7, 2016 at 10:31 AM, Ilia Mirkin wrote: >> On Wed, Dec 7, 2016 at 10:25 AM, Lionel Landwerlin >> wrote: >>> On 07/12/16 14:55, Ilia Mirkin wrote: On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin wrote: >

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 10:31 AM, Ilia Mirkin wrote: > On Wed, Dec 7, 2016 at 10:25 AM, Lionel Landwerlin > wrote: >> On 07/12/16 14:55, Ilia Mirkin wrote: >>> >>> On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin >>> wrote: Signed-off-by: Lionel Landwerlin --- src/compile

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 10:25 AM, Lionel Landwerlin wrote: > On 07/12/16 14:55, Ilia Mirkin wrote: >> >> On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin >> wrote: >>> >>> Signed-off-by: Lionel Landwerlin >>> --- >>> src/compiler/glsl/ast.h | 5 >>> src/compiler/glsl/a

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Lionel Landwerlin
On 07/12/16 14:55, Ilia Mirkin wrote: On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin wrote: Signed-off-by: Lionel Landwerlin --- src/compiler/glsl/ast.h | 5 src/compiler/glsl/ast_to_hir.cpp | 5 src/compiler/glsl/ast_type.cpp | 16 +++

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > src/compiler/glsl/ast.h | 5 > src/compiler/glsl/ast_to_hir.cpp | 5 > src/compiler/glsl/ast_type.cpp | 16 +++- > src/compiler/glsl/glsl_pa

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-01 Thread Chris Forbes
Ilia, it is different from the NV variant -- NV unfortunately threw a bunch of other stuff in (extra subpixel bits in conservative mode). Series is: Reviewed-by: Chris Forbes On Fri, Dec 2, 2016 at 8:21 AM, Ilia Mirkin wrote: > Ah, I see. The difference is that it exists, unlike the ARB one :

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-01 Thread Ilia Mirkin
Ah, I see. The difference is that it exists, unlike the ARB one :) I was confusing with the NV variant. On Dec 1, 2016 2:10 PM, "Ilia Mirkin" wrote: Is this different from the arb variant? On Dec 1, 2016 10:56 AM, "Lionel Landwerlin" wrote: > Signed-off-by: Lionel Landwerlin > --- > src/com

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-01 Thread Ilia Mirkin
Is this different from the arb variant? On Dec 1, 2016 10:56 AM, "Lionel Landwerlin" wrote: > Signed-off-by: Lionel Landwerlin > --- > src/compiler/glsl/ast.h | 5 > src/compiler/glsl/ast_to_hir.cpp | 5 > src/compiler/glsl/ast_type.cpp | 16 +

[Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-01 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/compiler/glsl/ast.h | 5 src/compiler/glsl/ast_to_hir.cpp | 5 src/compiler/glsl/ast_type.cpp | 16 +++- src/compiler/glsl/glsl_parser.yy | 34 ++ src/compiler/glsl/glsl