Signed-off-by: Chris Forbes
---
src/glsl/ast_function.cpp | 4 ++--
src/glsl/glsl_types.cpp | 3 ++-
src/glsl/glsl_types.h | 3 ++-
src/glsl/ir_function.cpp | 4 ++--
4 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/glsl/ast_function.cpp b/src/glsl/ast_function.cpp
index
V2: Fix crashes during linking, where the parse state is NULL. In this
case, all required checks have already been done, so we assume the
extension is enabled.
Signed-off-by: Chris Forbes
---
src/glsl/glsl_types.cpp | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff
The available implicit conversions depend on the GLSL version we're
compiling.
Signed-off-by: Chris Forbes
---
src/glsl/ir_function.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/glsl/ir_function.cpp b/src/glsl/ir_function.cpp
index 40cf589..4f0d9da 100644
--- a
Signed-off-by: Chris Forbes
---
docs/GL3.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 55906f2..7853724 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -104,7 +104,7 @@ GL 4.0:
- 'precise' qualifier
The ARB_gpu_shader5 spec says:
"A function definition A is considered a better
match than function definition B if:
* for at least one function argument, the conversion for that argument
in A is better than the corresponding conversion in B; and
* there is no function argument for which
We're about to add new implicit conversions, first for ARB_gpu_shader5,
and then later for ARB_gpu_shader_fp64. Pull out the opcode
determination into its own function, and get rid of the bool -> float
case that could never be hit anyway [since it fails the is_numeric()
check].
V2: Retain the vect
This is required for ARB_gpu_shader5.
Signed-off-by: Chris Forbes
---
src/glsl/ast_to_hir.cpp | 8
1 file changed, 8 insertions(+)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index b695132..f6c7409 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_hir.cpp
@@
This will facilitate GLSL 4.0 / ARB_gpu_shader5's enhanced overload
resolution rules, and also possibly better error reporting for ambiguous
function calls.
Signed-off-by: Chris Forbes
---
src/glsl/ir_function.cpp | 43 ---
1 file changed, 32 insertions(+)
This series adds support for implicit conversions and overload resolution from
ARB_gpu_shader5 / GLSL 4.0.
The first 5 patches (and half of the 9th) were previously sent out on their
own, but had some serious issues I had overlooked, as I hadn't done a full
piglit run.
Notable changes in this ver
The ARB_gpu_shader5 spec says:
"To determine whether the conversion for a single argument in one match is
better than that for another match, the following rules are applied, in
order:
1. An exact match is better than a match involving any implicit
conversion.
2. A match involving an im
Gallium already supports stencil texturing and st/mesa has been using
it for glDrawPixels for quite a while. The pipe_sampler_view bit is
unnecessary. You can create a stencil sampler view by specifying one
of these formats:
PIPE_FORMAT_S8_UINT
PIPE_FORMAT_X24S8_UINT
PIPE_FORMAT_S8X24_UINT
PIPE_FO
Yeah, I actually sent a follow-up patch which does this in a totally
different and much simpler way.
On Sun, May 4, 2014 at 6:47 AM, Marek Olšák wrote:
> Gallium already supports stencil texturing and st/mesa has been using
> it for glDrawPixels for quite a while. The pipe_sampler_view bit is
> u
Reviewed-by: Marek Olšák
Marek
On Fri, May 2, 2014 at 5:00 PM, pstglia wrote:
> ---
> src/gallium/state_trackers/egl/android/native_android.cpp | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git a/src/gallium/state_trackers/egl/android/native_android.cpp
> b/src/gallium/state_track
One question: Should we have a list of drivers supporting a
gpu_shader5 feature after "DONE" like the other extensions have? For
features without driver changes, that would be "DONE (all drivers)".
Marek
On Sun, May 4, 2014 at 10:24 AM, Chris Forbes wrote:
> Signed-off-by: Chris Forbes
> ---
>
Yes, we probably should. Some of the other features require a
significant amount of hardware-specific work.
On Sun, May 4, 2014 at 11:11 PM, Marek Olšák wrote:
> One question: Should we have a list of drivers supporting a
> gpu_shader5 feature after "DONE" like the other extensions have? For
> fe
Reviewed-by: Marek Olšák
Marek
On Sat, May 3, 2014 at 11:25 AM, Ilia Mirkin wrote:
> If StencilSampling is enabled on the texture object, pass in an
> equivalent stencil-only format.
>
> Signed-off-by: Ilia Mirkin
> ---
>
> This replaces my earlier series, based on a suggestion from Michel Dän
I think the plan was to add "struct pipe_query *render_condition" to
pipe_blit_info. The value of NULL would mean the conditional rendering
is disabled.
Marek
On Sat, May 3, 2014 at 10:42 AM, Ilia Mirkin wrote:
> Hello,
>
> I've noticed that nv50 fails the latest version of the conditional
> ren
https://bugs.freedesktop.org/show_bug.cgi?id=78258
Priority: medium
Bug ID: 78258
Keywords: regression
Assignee: mesa-dev@lists.freedesktop.org
Summary: make check link_varyings.gl_ClipDistance failure
Severity: major
Classificatio
Am 03.05.2014 11:25, schrieb Ilia Mirkin:
> If StencilSampling is enabled on the texture object, pass in an
> equivalent stencil-only format.
>
> Signed-off-by: Ilia Mirkin
> ---
>
> This replaces my earlier series, based on a suggestion from Michel Dänzer on
> IRC. Much simpler, and enables it
Why would you need the whole query? A boolean if it should honor the
currently set render condition sounds simpler and good enough to me.
Roland
Am 04.05.2014 13:17, schrieb Marek Olšák:
> I think the plan was to add "struct pipe_query *render_condition" to
> pipe_blit_info. The value of NULL wou
https://bugs.freedesktop.org/show_bug.cgi?id=78258
Vinson Lee changed:
What|Removed |Added
CC||i...@freedesktop.org
--- Comment #1 from Vi
Yes, you are right, a boolean flag would be simpler.
Marek
On Sun, May 4, 2014 at 6:06 PM, Roland Scheidegger wrote:
> Why would you need the whole query? A boolean if it should honor the
> currently set render condition sounds simpler and good enough to me.
>
> Roland
>
> Am 04.05.2014 13:17, s
On Sat, May 3, 2014 at 1:52 AM, Ian Romanick wrote:
> On 04/22/2014 01:58 AM, Chia-I Wu wrote:
>> There may be two contexts compiling shaders at the same time. locale_t needs
>> to be protected.
>
> Rather than calling glsl_initialize_strtod from other places in the
> compiler, it seems better to
On Sat, May 3, 2014 at 1:33 AM, Ian Romanick wrote:
> On 04/22/2014 01:58 AM, Chia-I Wu wrote:
>> There may be two contexts compiling shaders at the same time, and we want the
>> anonymous struct id to be globally unique.
>
> I am not very excited about this.
>
> Is there any chance of getting std
On Sat, May 3, 2014 at 1:59 AM, Ian Romanick wrote:
> On 04/22/2014 01:58 AM, Chia-I Wu wrote:
>> From: Chia-I Wu
>>
>> Threaded glCompileShader can be enabled for a context by calling
>> _mesa_enable_glsl_threadpool. It will initialize the singleton GLSL thread
>> pool and defer glCompileShader
On Sun, May 4, 2014 at 7:02 PM, Marek Olšák wrote:
> Reviewed-by: Marek Olšák
Looks good to me too, except please use your real name.
> Marek
>
> On Fri, May 2, 2014 at 5:00 PM, pstglia wrote:
>> ---
>> src/gallium/state_trackers/egl/android/native_android.cpp | 7 +++
>> 1 file changed, 7
On Fri, May 2, 2014 at 10:33 AM, Ian Romanick wrote:
> On 04/22/2014 01:58 AM, Chia-I Wu wrote:
>> There may be two contexts compiling shaders at the same time, and we want the
>> anonymous struct id to be globally unique.
>
> I am not very excited about this.
>
> Is there any chance of getting st
On Sun, May 4, 2014 at 11:59 AM, Roland Scheidegger wrote:
> Am 03.05.2014 11:25, schrieb Ilia Mirkin:
>> If StencilSampling is enabled on the texture object, pass in an
>> equivalent stencil-only format.
>>
>> Signed-off-by: Ilia Mirkin
>> ---
>>
>> This replaces my earlier series, based on a su
On Mon, May 5, 2014 at 6:12 AM, Paulo Sergio wrote:
> Ok. Name is Paulo Sergio Travaglia. Shall I resend the patch?
I've committed it. Thanks.
> Thanks
>
> Em 04/05/2014 18:40, "Chia-I Wu" escreveu:
>
>> On Sun, May 4, 2014 at 7:02 PM, Marek Olšák wrote:
>> > Reviewed-by: Marek Olšák
>> Looks
Patch adds new implementation dependent value required by the
GL_ARB_explicit_uniform_location extension. Default value for user
assignable locations is calculated as sum of MaxUniformComponents
for each stage.
v2: fix descriptor in get_hash_params.py (Petri)
Signed-off-by: Tapani Pälli
---
src
Print out GL_ARB_explicit_attrib_location warnings only
when parsing attribute that uses "location" qualifier.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77245
Signed-off-by: Tapani Pälli
---
src/glsl/glsl_parser.yy | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78101
Signed-off-by: Tapani Pälli
---
src/egl/opengles1/drawtex.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/egl/opengles1/drawtex.c b/src/egl/opengles1/drawtex.c
index 524f931..1ba8ac7 100644
--- a/src/
https://bugs.freedesktop.org/show_bug.cgi?id=78101
Tapani Pälli changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|mesa-dev@list
https://bugs.freedesktop.org/show_bug.cgi?id=73512
Igor Gnatenko changed:
What|Removed |Added
Attachment #92091|0 |1
is obsolete|
34 matches
Mail list logo
