On 06/10/2014 10:20 PM, Ian Romanick wrote:
I did a little more research. Looking at the commit history, it must
just be an MSVC thing:
commit 78d3cfb5b4fe1dbe052f87458e62cc2be2ecf171
Author: Brian Paul
Date: Sun Nov 4 16:43:44 2012 -0700
glsl: remove incorrect 'struct' keyword
i
---
src/glsl/link_varyings.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/glsl/link_varyings.h b/src/glsl/link_varyings.h
index 4e485c3..167363e 100644
--- a/src/glsl/link_varyings.h
+++ b/src/glsl/link_varyings.h
@@ -112,6 +112,16 @@ public:
return !this->next_buffer
Outputs that are linked to inputs in the next stage must be output to stream 0,
otherwise we should fail to link.
---
src/glsl/link_varyings.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 9883c0b..0066b4e 100644
--- a/sr
If this is detected, fail to link.
---
src/glsl/link_varyings.cpp | 17 +
1 file changed, 17 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 0066b4e..1a69897 100644
--- a/src/glsl/link_varyings.cpp
+++ b/src/glsl/link_varyings.cpp
@@ -635,1
On Intel hardware when a geometry shader outputs GL_POINTS primitives we
only need to emit vertex control bits if it emits vertices to non-zero
streams, so use a flag to track this.
This flag will be set to TRUE when a geometry shader calls EmitStreamVertex()
or EndStreamPrimitive() with a non-zer
Inter-shader outputs must be on stream 0, which is the default.
---
src/glsl/link_varyings.cpp | 12 +---
src/glsl/link_varyings.h | 7 +++
2 files changed, 16 insertions(+), 3 deletions(-)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index f765d37..9883c0b
>From the ARB_gpu_shader5 spec:
"Multiple vertex streams are supported only if the output primitive
type is declared to be "points". A program will fail to link if it
contains a geometry shader calling EmitStreamVertex() or
EndStreamPrimitive() if its output primitive type is not "poin
For now initialized to the default stream 0.
---
src/glsl/link_varyings.cpp | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 6863298..f765d37 100644
--- a/src/glsl/link_varyings.cpp
+++ b/src/glsl/
Configure hardware to read vertex data for all streams and have all streams
write their varyings to the corresponsing output buffers.
---
src/mesa/drivers/dri/i965/gen7_sol_state.c | 65 +++---
1 file changed, 41 insertions(+), 24 deletions(-)
diff --git a/src/mesa/drivers
From: Samuel Iglesias Gonsalvez
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/glsl/ir_print_visitor.cpp | 7 +--
src/glsl/ir_reader.cpp| 6 ++
2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/src/glsl/ir_print_visitor.cpp b/src/glsl/ir_print_visitor.cpp
index 2
This should be ensured by the parser, so assert on that.
---
src/glsl/link_varyings.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 1a69897..aed069a 100644
--- a/src/glsl/link_varyings.cpp
+++ b/src/glsl/link_varyings.cpp
@
---
src/glsl/ast_function.cpp | 37 +-
src/glsl/builtin_functions.cpp | 60 ++
src/glsl/ir.h | 18 -
3 files changed, 103 insertions(+), 12 deletions(-)
diff --git a/src/glsl/ast_function.cpp b/src/g
It should be possible to query the number of primitives written to each
individual stream by a geometry shader in a single draw call. For that
we need to have up to MAX_VERTEX_STREAM separate query objects.
---
src/mesa/main/mtypes.h | 2 +-
src/mesa/main/queryobj.c | 17 +
2 fi
---
src/mesa/main/context.c | 2 +-
src/mesa/main/mtypes.h | 1 +
src/mesa/main/queryobj.c | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index 8eb426d..82b00eb 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/conte
This series brings multi-stream support to geometry shaders as defined by
ARB_gpu_shader5. It also covers some missing multi-stream functionality from
ARB_transform_feedback3. This is combined work from Samuel Iglesias and
myself.
The series includes both required infrastructure in Mesa and specif
If the geometry shader is indeed using streams then we need 2 control data
bits per vertex for the StreamID. If the shader is not using streams then
we don't need control data bits.
---
src/mesa/drivers/dri/i965/brw_vec4_gs.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --g
---
src/glsl/ir_print_visitor.cpp | 8
src/glsl/ir_reader.cpp| 18 ++
2 files changed, 18 insertions(+), 8 deletions(-)
diff --git a/src/glsl/ir_print_visitor.cpp b/src/glsl/ir_print_visitor.cpp
index c4a6f9c..23960e0 100644
--- a/src/glsl/ir_print_visitor.cpp
++
From: Samuel Iglesias Gonsalvez
This implements parsing requirements for multi-stream support in
geometry shaders as defined in ARB_gpu_shader5.
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/glsl/ast.h | 5 +
src/glsl/ast_to_hir.cpp | 6 ++
src/glsl/ast_type.cpp | 19 ++
---
src/mesa/drivers/dri/i965/gen6_queryobj.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen6_queryobj.c
b/src/mesa/drivers/dri/i965/gen6_queryobj.c
index 06ac645..0cb64ca 100644
--- a/src/mesa/drivers/dri/i965/gen6_queryobj.c
+++ b/src/m
In stream mode we have to set control data bits with the StreamID
information for every vertex.
---
src/mesa/drivers/dri/i965/brw_vec4_gs_visitor.cpp | 51 ++-
src/mesa/drivers/dri/i965/brw_vec4_gs_visitor.h | 1 +
2 files changed, 51 insertions(+), 1 deletion(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=79907
Priority: medium
Bug ID: 79907
Assignee: mesa-dev@lists.freedesktop.org
Summary: Mesa 10.2.1 --enable-vdpau default=auto broken
Severity: normal
Classification: Unclassified
On Wed, Jun 11, 2014 at 7:49 PM, Iago Toral Quiroga wrote:
> @@ -509,6 +509,8 @@ struct ast_type_qualifier {
> /** \name Layout qualifiers for GL_ARB_gpu_shader5 */
> /** \{ */
> unsigned invocations:1;
> + unsigned streamId:1; /* Has streamId value assigned
This would appear to prohibit the use of multiple streams in separable
programs entirely. I don't think that's the right thing.
On Wed, Jun 11, 2014 at 7:49 PM, Iago Toral Quiroga wrote:
> Outputs that are linked to inputs in the next stage must be output to stream
> 0,
> otherwise we should fai
I think you need to set this if *any* geometry shader in the program
tries to use a nonzero stream. Currently it looks like the last shader
wins.
On Wed, Jun 11, 2014 at 7:49 PM, Iago Toral Quiroga wrote:
> On Intel hardware when a geometry shader outputs GL_POINTS primitives we
> only need to em
This is pretty weird.
We should be able to generate a normal builtin function body here
which consists of just the ir_emit_vertex, passing the stream
parameter to it. This would then get inlined like any other function
leaving a bare ir_emit_vertex / ir_end_primitive with a constant
operand. If on
On Wed, Jun 11, 2014 at 7:49 PM, Iago Toral Quiroga wrote:
> ---
> src/mesa/main/context.c | 2 +-
> src/mesa/main/mtypes.h | 1 +
> src/mesa/main/queryobj.c | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
> index 8eb4
On Wed, Jun 11, 2014 at 7:49 PM, Iago Toral Quiroga wrote:
> It should be possible to query the number of primitives written to each
> individual stream by a geometry shader in a single draw call. For that
> we need to have up to MAX_VERTEX_STREAM separate query objects.
> ---
> src/mesa/main/mty
I sent comments on patches 1, 3, 5, 9, 11, 16, 18
Patches 2, 4, 6-8, 10, 12-15, 17 are:
Reviewed-by: Chris Forbes
You should also include a patch to docs/GL3.txt marking off this
subfeature for i965 :)
Do you have a bunch of piglits which exercise all the tricky corners
of this? I see a few t
Hi Chris,
thanks for the quick review!
On Wed, 2014-06-11 at 21:45 +1200, Chris Forbes wrote:
> I sent comments on patches 1, 3, 5, 9, 11, 16, 18
We will look into these.
> Patches 2, 4, 6-8, 10, 12-15, 17 are:
>
> Reviewed-by: Chris Forbes
>
> You should also include a patch to docs/GL3.txt
I am a bit confused here:
Reading the code it looks like input_var should be NULL if there is no
consumer stage. In that case, if this is a separable program and there
is no consumer, then input_var must be NULL too, in which case the
linker_error would never take place.
If input_var is not NULL
https://bugs.freedesktop.org/show_bug.cgi?id=79885
Marc Dietrich changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Piglit report an error in glsl-link-initializer-03 test when
a shader without an initializer is linked in two different programs with
shaders that have differing initializers. Linking with the first will
modify the shader, and linking with the second will fail.
To make it work, the patch detects
Hi,
Thank you very much.
On Tue, Jun 10, 2014 at 5:28 PM, Marek Olšák wrote:
> On Tue, Jun 10, 2014 at 10:45 PM, Aditya Avinash
> wrote:
> > Hi,
> > Thank you very much!!
> >
> > On Mon, Jun 9, 2014 at 10:19 AM, Marek Olšák wrote:
> >>
> >> This is probably one of the most difficult tasks. Yo
That commit made possible that the items could be one just
after the other when their size was a multiple of ITEM_ALIGNMENT.
But compute_memory_prealloc_chunk still looked to leave a gap
between items. Resulting in that we got an infinite loop when
trying to add an item which would left no space be
Backport last round of piglit fixes to 10.2. These have been soaking
on master for a couple weeks, and I've not found any regression yet.
And they get us from 80% pass up to nearly 83% pass, so a bit more
progress in the right direction. (For reference, at end of Feb we
were at 50% pass ;-))
Als
On Wed, Jun 11, 2014 at 2:49 AM, Iago Toral wrote:
> Hi Chris,
>
> thanks for the quick review!
>
> On Wed, 2014-06-11 at 21:45 +1200, Chris Forbes wrote:
>> I sent comments on patches 1, 3, 5, 9, 11, 16, 18
>
> We will look into these.
>
>> Patches 2, 4, 6-8, 10, 12-15, 17 are:
>>
>> Reviewed-by:
Thanks a lot for your help.!
CFLAGS='-O0 -ggdb3' CXXFLAGS='-O0 -ggdb3' ./autogen.sh --prefix=/usr
--libdir=/usr/lib/i386-linux-gnu --with-gallium-drivers=svga
--with-dri-drivers= --enable-xa --disable-dri3; make;
did the trick.
On Thu, Jun 5, 2014 at 9:01 AM, Ian Romanick wrote:
> On 06/04/20
Adding mesa-dev in CC.
On Wed, Jun 11, 2014 at 10:39 AM, Courtney Goeltzenleuchter <
court...@lunarg.com> wrote:
>
>
>
> On Fri, Jun 6, 2014 at 5:57 PM, Anuj Phogat wrote:
>
>> Fixes many failures in gles3 Khronos CTS test: packed_pixels
>>
>> Khronos bug# 9807
>> Cc:
>> Signed-off-by: Anuj Ph
Adding mesa-dev to the conversation.
On Wed, Jun 11, 2014 at 10:52 AM, Anuj Phogat wrote:
>
> On Wed, Jun 11, 2014 at 10:28 AM, Courtney Goeltzenleuchter
> wrote:
>>
>> On Fri, Jun 6, 2014 at 5:57 PM, Anuj Phogat wrote:
>>>
>>> Fixes many failures in gles3 Khronos CTS test: packed_pixels
>>>
>>
Very good. I thought there might be something limiting the inputs but
wasn't immediately obvious.
On Wed, Jun 11, 2014 at 12:09 PM, Anuj Phogat wrote:
> Adding mesa-dev to the conversation.
>
> On Wed, Jun 11, 2014 at 10:52 AM, Anuj Phogat
> wrote:
> >
> > On Wed, Jun 11, 2014 at 10:28 AM, Cou
On Tue, Jun 10, 2014 at 3:34 PM, Matt Turner wrote:
> On Wed, May 28, 2014 at 4:44 PM, Connor Abbott wrote:
>> On Tue, May 27, 2014 at 9:47 PM, Matt Turner wrote:
>>> Here's a respin of my load_payload series from mid-April with some
>>> feedback from Ken addressed and some bugs fixed.
>>>
>>> T
GL_TEXTURE_CUBE_MAP is an allowed texture target in glTexStorage2D()
and is allowed to be used (like GL_TEXTURE_2D) with compressed internal
formats.
Cc:
Signed-off-by: Anuj Phogat
---
src/mesa/main/teximage.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/teximage.c b/src/me
Too many levels of indirection.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp
b/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp
index aeed6c9..064a5
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
I had misread it -- yes, this looks correct.
On Thu, Jun 12, 2014 at 12:02 AM, Iago Toral wrote:
> I am a bit confused here:
>
> Reading the code it looks like input_var should be NULL if there is no
> consumer stage. In that case, if this is a separable program and there
> is no consumer, then i
Reviewed-by: Marek Olšák
Marek
On Wed, Jun 11, 2014 at 9:02 PM, Anuj Phogat wrote:
> GL_TEXTURE_CUBE_MAP is an allowed texture target in glTexStorage2D()
> and is allowed to be used (like GL_TEXTURE_2D) with compressed internal
> formats.
>
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa
On 10/06/14 16:56, Niels Ole Salscheider wrote:
> On Tuesday 10 June 2014, 16:18:56, Emil Velikov wrote:
>> On 10/06/14 15:17, Niels Ole Salscheider wrote:
>>> This fixes automake builds which are broken since
>>> b52a530ce2aada1967bc8fefa83ab53e6a737dae.
>>
>> Not sure what I was smoking with the
https://bugs.freedesktop.org/show_bug.cgi?id=79688
Eugene Shalygin changed:
What|Removed |Added
CC||eugene.shalygin+bugzilla.FD
On Tue, Jun 10, 2014 at 6:00 PM, Ian Romanick wrote:
> On 06/09/2014 07:48 AM, Robert Bragg wrote:
>> This makes sure to use a no-op swizzle while iteratively rendering each
>> level of a mipmap otherwise we may loose components and effectively
>> apply the swizzle twice by the time these levels a
On Tue, Jun 10, 2014 at 09:57:20PM -0700, Matt Turner wrote:
> From: Kenneth Graunke
>
> This allows GCC (at least with LTO) to eliminate more unused code, such
> as brw_clip* on Gen6+.
If we compile with STATIC_GEN, then the if-else ladder in brw_init_state()
turns into just the one case that m
This fixes automake builds which are broken since
b52a530ce2aada1967bc8fefa83ab53e6a737dae.
v2: This patch also adds the FEATURE_* defines back to targets/egl-static for
Android and Scons that have been removed in the mentioned commit.
Signed-off-by: Niels Ole Salscheider
Bugzilla: https://bugs.
On Tue, Jun 10, 2014 at 09:57:17PM -0700, Matt Turner wrote:
Except patch 4, series
Reviewed-by: Kristian Høgsberg
> ---
> src/mesa/main/macros.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/macros.h b/src/mesa/main/macros.h
> index 5228c3a..df6cb01
On Tue, Jun 10, 2014 at 07:37:42PM -0700, Matt Turner wrote:
> Please stop adding them.
Reviewed-by: Kristian Høgsberg
> ---
> configure.ac | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index eaca378..c6664b9 100644
> --- a/configure
On Tue, Jun 10, 2014 at 07:37:43PM -0700, Matt Turner wrote:
> This isn't supposed to be difficult.
Snarky-snark.
Reviewed-by: Kristian Høgsberg
> ---
> configure.ac | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index c66
On Tue, Jun 10, 2014 at 07:37:44PM -0700, Matt Turner wrote:
Reviewed-by: Kristian Høgsberg
> ---
> configure.ac | 12 +++-
> 1 file changed, 3 insertions(+), 9 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 17e9e9c..4b27455 100644
> --- a/configure.ac
> +++ b/confi
On Wed, Jun 11, 2014 at 12:59:21PM +0900, Michel Dänzer wrote:
> From: Michel Dänzer
>
> GBM_BO_USE_CURSOR_64X64 is kept so that existing users of GBM continue to
> build, but it no longer rejects widths or heights other than 64.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79809
>
On Tue, Jun 10, 2014 at 11:21:26AM -0600, Courtney Goeltzenleuchter wrote:
> The code that parses LIBGL_DRIVERS_PATH was printing an
> error for every attempted dlopen. It's not an error to
> have to check multiple items in the path, only an error if
> no suitable library is found. Reduced the load
Commit e62b7d38a1d (configure: autodetect video state-trackers
when non swrast driver is present) added a check that caused
the autodetection to be omitted when we have the swrast gallium
driver. Whereas it should have skipped the VL targets when only
swrast was selected.
Bugzilla: https://bugs.fr
https://bugs.freedesktop.org/show_bug.cgi?id=79907
--- Comment #1 from Emil Velikov ---
Thanks for reporting, a silly thinko on my behalf is causing this behaviour.
Can you confirm that the patch [1] fixes the issue on your system ?
[1] http://patchwork.freedesktop.org/patch/27562/
--
You are
On 11/06/14 03:37, Matt Turner wrote:
> Please stop adding them.
Believe I contributed the latter one :)
The series looks good imho
Reviewed-by: Emil Velikov
> ---
> configure.ac | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index ea
On Wed, Jun 11, 2014 at 2:12 PM, Kristian Høgsberg wrote:
> On Tue, Jun 10, 2014 at 09:57:20PM -0700, Matt Turner wrote:
>> From: Kenneth Graunke
>>
>> This allows GCC (at least with LTO) to eliminate more unused code, such
>> as brw_clip* on Gen6+.
>
> If we compile with STATIC_GEN, then the if-
On Tue, Jun 10, 2014 at 11:00:43AM -0700, Matt Turner wrote:
> Our most frequently used list implementation was only usable from
> C by manipulating the members of the structs. This series adds a
> C API, wraps the C API with the C++ member functions (so no change
> for C++ code), makes the foreach
On 06/11/2014 01:40 AM, Chris Forbes wrote:
> On Wed, Jun 11, 2014 at 7:49 PM, Iago Toral Quiroga wrote:
>> @@ -509,6 +509,8 @@ struct ast_type_qualifier {
>> /** \name Layout qualifiers for GL_ARB_gpu_shader5 */
>> /** \{ */
>> unsigned invocations:1;
>> + un
On 06/11/2014 12:49 AM, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> This implements parsing requirements for multi-stream support in
> geometry shaders as defined in ARB_gpu_shader5.
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/glsl/ast.h | 5 +
>
From: Marek Olšák
This adds a new type of intrinsic and SDNode: SampleRaw.
All fields of the MIMG opcodes are exposed and can be set by Mesa,
even DMASK. All GATHER4 variants are added and there are a lot of them.
v2: document DMASK behavior
---
lib/Target/R600/AMDGPUISelLowering.cpp | 24 +
From: Marek Olšák
---
test/CodeGen/R600/llvm.SI.gather4.ll | 508 +++
1 file changed, 508 insertions(+)
create mode 100644 test/CodeGen/R600/llvm.SI.gather4.ll
diff --git a/test/CodeGen/R600/llvm.SI.gather4.ll
b/test/CodeGen/R600/llvm.SI.gather4.ll
new file mod
Anuj Phogat writes:
> Fixes piglit tests in spec/glsl-es-3.00/compile:
> undef-__FILE__.vert
> undef-GL_ES.vert
> undef-__LINE__.vert
> undef-__VERSION__.vert
I see the code that will fix __FILE__, __LINE__, and __VERSION__, but
not GL_ES. Should that line be dropped from the commit message?
> +
Currently verifying that an #undef of __FILE__, __LINE__, or __VERSION__ will
generate an error.
---
src/glsl/glcpp/tests/120-undef-builtin.c | 3 +++
src/glsl/glcpp/tests/120-undef-builtin.c.expected | 7 +++
2 files changed, 10 insertions(+)
create mode 100644 src/glsl/glcpp/tests/
I've been looking at this is a bit since last week, and I have some
concerns about the overall way things are structured. There are also
almost no checks that the values being de-serialized are valid. What
happens if an ir_expression::operation is 0xDEADBEEF? I think I know
where we want to be f
On Wed, Jun 11, 2014 at 5:32 PM, Carl Worth wrote:
> Anuj Phogat writes:
>> Fixes piglit tests in spec/glsl-es-3.00/compile:
>> undef-__FILE__.vert
>> undef-GL_ES.vert
>> undef-__LINE__.vert
>> undef-__VERSION__.vert
>
> I see the code that will fix __FILE__, __LINE__, and __VERSION__, but
> not
Anuj Phogat writes:
> This patch specifically fixes redefinition condition for white space
> changes. #define and #undef functionality in GLSL follows the standard
> for C++ preprocessors for macro definitions.
Thanks! That fix took some close reading of the specification, and some
tweaking to so
https://bugs.freedesktop.org/show_bug.cgi?id=54372
--- Comment #22 from danm...@gmail.com ---
Ok, so this patch fixes glxUseXFont for glxwindows.
Any idea when these patches will go in?
diff --git a/src/glx/xfont.c b/src/glx/xfont.c
index 316c585..3c34b1d 100644
--- a/src/glx/xfont.c
+++ b/src/
Previously, the test suite was expecting the compiler to allow a redefintion
of a macro with whitespace added, but gcc is more strict and allows only for
changes in the amounts of whitespace, (but insists that whitespace exist or
not in exactly the same places).
See: https://gcc.gnu.org/onlinedocs
Anuj Phogat writes:
>> #if 0
>> #else garbage
>> #endif
>>
>> The Khronos test contains two tokens after #else. I'm not sure if that
>> is the difference or not.
> Khronos is testing many different cases. Failing case tests:
>
> #if 1
> #else garbage
> #endif
It is the "#if 1" that causes the exi
On Wed, Jun 11, 2014 at 6:51 PM, Carl Worth wrote:
> Anuj Phogat writes:
> >> #if 0
> >> #else garbage
> >> #endif
> >>
> >> The Khronos test contains two tokens after #else. I'm not sure if that
> >> is the difference or not.
> > Khronos is testing many different cases. Failing case tests:
> >
Anuj Phogat writes:
> Following changes in above code will also fix the GL_ES macro case:
> - || strcmp("__VERSION__", $3) == 0)
> + || strcmp("__VERSION__", $3) == 0
> + || (parser->is_gles && strcmp("GL_ES", $3) == 0))
Great.
> Do you want
Carl Worth writes:
> I'll poke around at the existing code to see why the garbage after #else
> is caught with #if 0, but not with #if 1.
The difference is fairly straightforward, but it's going to be tricky to
fix.
Currently, the parser has a production for HASH_ELSE and NEWLINE tokens
that loo
Without this, in the state, we would hit Flex's default rule, which
prints tokens to stdout, rather than returning them as tokens.
This manifested as a weird bug where shaders with semicolons after
extension directives, such as:
#extension GL_foo_bar : enable;
would print semicolons to the s
Ian Romanick writes:
> On 06/09/2014 12:36 AM, Kenneth Graunke wrote:
>> We have this wonderful thing called an optimizer---maybe we should
>> consider calling it?
>
> That's really weird. The *only* place this is called is in ir_to_mesa.
> It seems like the assember should just call it after a
Hi;
On 06/02/2014 03:05 PM, Tapani Pälli wrote:
> Hello;
>
> This series provides a disk cache for the shader compiler and is used
> to 'skip glLinkProgram' like GL_ARB_get_program_binary does but under
> the hood without api for the client.
>
>
> With the series I get 50% improvement for L4D2 s
How did you notice this? (More specifically than semicolons being
printed in your terminals!)
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Am 11.06.2014 23:46, schrieb Emil Velikov:
Commit e62b7d38a1d (configure: autodetect video state-trackers
when non swrast driver is present) added a check that caused
the autodetection to be omitted when we have the swrast gallium
driver. Whereas it should have skipped the VL targets when only
sw
On Wed, 2014-06-11 at 15:29 -0700, Ian Romanick wrote:
> On 06/11/2014 12:49 AM, Iago Toral Quiroga wrote:
> > From: Samuel Iglesias Gonsalvez
> >
> > This implements parsing requirements for multi-stream support in
> > geometry shaders as defined in ARB_gpu_shader5.
> >
> > Signed-off-by: Samue
It may be a bit less confusing to read if I take the inner if out, so
I'll rewrite it like this:
if (input_var || (prog->SeparateShader && consumer == NULL)) {
matches.record(output_var, input_var);
}
if (input_var && output_var->data.streamId != 0) {
linker_error(prog, ...);
return fals
On 06/05/2014 08:08 AM, Tapani Pälli wrote:
Patch adds a preprocessor define for the extension and stores explicit
location data for uniforms during AST->HIR conversion. It also sets
layout token to be available when having the extension in place.
v2: change parser check to require GLSL 330 or
On 06/12/2014 09:44 AM, Petri Latvala wrote:
> On 06/05/2014 08:08 AM, Tapani Pälli wrote:
>> Patch adds a preprocessor define for the extension and stores explicit
>> location data for uniforms during AST->HIR conversion. It also sets
>> layout token to be available when having the extension in pl
86 matches
Mail list logo
