On Thursday, April 21, 2016 10:18:48 PM PDT Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/Makefile.sources | 1 +
> src/mesa/drivers/dri/i965/brw_context.c | 4 +-
> src/mesa/drivers/dri/i965/brw_context.h | 5 +
> src/mesa/drivers/dri/i96
On 26/04/16 06:39, Jason Ekstrand wrote:
> This commit adds a validator that ensures that all expressions passed
> through nir_algebraic are 100% non-ambiguous as far as bit-sizes are
> concerned. This way it's a compile-time error rather than a hard-to-trace
> C exception some time later.
> ---
Patches 1, 4 and 5 need rebasing.
Patches 2, 3 and 6 are:
Reviewed-by: Eduardo Lima Mitev
Thanks for fixing this! it has annoyed me too.
Eduardo
On 04/27/2016 05:41 AM, Jason Ekstrand wrote:
> I think things started out consistent when we first merged NIR since there
> weren't a lot of helper
Reviewed-by: Eduardo Lima Mitev
Thanks!
On 04/27/2016 05:20 AM, Timothy Arceri wrote:
> ---
> src/compiler/glsl/link_uniform_blocks.cpp | 53
> +++
> src/compiler/glsl/link_uniforms.cpp | 53
> ---
> 2 files changed, 53 insertions(
This matches the "foreach x in container" pattern found in many other
programming languages. Generated by the following regular expression:
s/nir_foreach_def(\([^,]*\),\s*\([^,]*\))/nir_foreach_def(\2, \1)/
---
src/compiler/nir/nir.h | 4 ++--
src/compiler/nir/nir_opt_global_
This matches the "foreach x in container" pattern found in many other
programming languages.
---
src/compiler/nir/nir.c | 4 ++--
src/compiler/nir/nir.h | 2 +-
src/compiler/nir/nir_from_ssa.c | 6 +++---
src/compiler/nir/nir_print.c| 2 +-
4 files changed, 7 insertions(+), 7
This matches the "foreach x in container" pattern found in many other
programming languages. Generated by the following regular expression:
s/nir_foreach_use(\([^,]*\),\s*\([^,]*\))/nir_foreach_use(\2, \1)/
and similar expressions for nir_foreach_use_safe, etc.
---
src/compiler/nir/nir.c
This matches the "foreach x in container" pattern found in many other
programming languages. Generated by the following regular expression:
s/nir_foreach_instr(\([^,]*\),\s*\([^,]*\))/nir_foreach_instr(\2, \1)/
and similar expressions for nir_foreach_instr_safe etc.
---
src/compiler/nir/nir.c
This matches the "foreach x in container" pattern found in many other
programming languages. Generated by the following regular expression:
s/nir_foreach_phi_src(\([^,]*\),\s*\([^,]*\))/nir_foreach_phi_src(\2, \1)/
and a similar expression for nir_foreach_phi_src_safe.
---
src/compiler/nir/nir.
This matches the "foreach x in container" pattern found in many other
programming languages. Generated by the following regular expression:
s/nir_foreach_function(\([^,]*\),\s*\([^,]*\))/nir_foreach_function(\2, \1)/
---
src/compiler/nir/nir.h | 2 +-
src/c
I think things started out consistent when we first merged NIR since there
weren't a lot of helpers. Over time, however, more nir_foreach helpers
have been added and they haven't had a consistent argument order. This has
been annoying me for a while ("Which way does this one work again?") and
thi
---
src/compiler/glsl/link_uniform_blocks.cpp | 53 +++
src/compiler/glsl/link_uniforms.cpp | 53 ---
2 files changed, 53 insertions(+), 53 deletions(-)
diff --git a/src/compiler/glsl/link_uniform_blocks.cpp
b/src/compiler/glsl/link_u
On Tuesday, April 26, 2016 1:08:02 PM PDT Jamey Sharp wrote:
> OpenGL 4.5 Core Profile section 7.1, in the documentation for
> CompileShader, says: "Changing the source code of a shader object with
> ShaderSource does not change its compile status or the compiled shader
> code." (I haven't checked
On Monday, April 25, 2016 10:06:40 PM PDT Jamey Sharp wrote:
> OpenGL 4.5 Core Profile section 7.1, in the documentation for
> CompileShader, says: "Changing the source code of a shader object with
> ShaderSource does not change its compile status or the compiled shader
> code." (I haven't checked
On Monday, April 25, 2016 5:54:28 PM PDT Jordan Justen wrote:
> Fixes the OpenGLES 3.1 CTS:
> * ESEXT-CTS.draw_elements_base_vertex_tests.invalid_mapped_bos
>
> Because this is triggering the error message after the normal API
> validation phase, we don't have the API function name available, and
On Monday, April 25, 2016 5:54:28 PM PDT Jordan Justen wrote:
> Fixes the OpenGLES 3.1 CTS:
> * ESEXT-CTS.draw_elements_base_vertex_tests.invalid_mapped_bos
>
> Because this is triggering the error message after the normal API
> validation phase, we don't have the API function name available, and
Am 27.04.2016 um 03:05 schrieb Dave Airlie:
> On 27 April 2016 at 11:00, Dave Airlie wrote:
So far I've set the execmask to 1 active channel, I'm contemplating
changing that
though and using less machines.
>>> Ah yes, I think that would indeed be desirable.
>>
>> I'll look into it,
Thanks again, I will make/modify those piglit tests.
Regards,
Lars Hamre
On Tue, Apr 26, 2016 at 9:20 PM, Timothy Arceri
wrote:
> On Tue, 2016-04-26 at 19:50 -0400, Lars Hamre wrote:
>> v2: limit lowerings to return statments in main()
>>
>> Return statements in conditional blocks were not havin
On Tue, 2016-04-26 at 19:50 -0400, Lars Hamre wrote:
> v2: limit lowerings to return statments in main()
>
> Return statements in conditional blocks were not having their
> output varyings lowered correctly.
>
> This patch fixes the following piglit tests:
> /spec/glsl-1.10/execution/vs-float-mai
Good catch!
Reviewed-by: Jason Ekstrand
Why does it cause an assert?
On Tue, Apr 26, 2016 at 3:35 PM, Nanley Chery wrote:
> From: Nanley Chery
>
> Fixes some failures in dEQP-VK.api.info.image_format_properties.* and
> enables the test group to execute without assert failing.
>
> Bugzilla: h
On 27 April 2016 at 11:00, Dave Airlie wrote:
>>> So far I've set the execmask to 1 active channel, I'm contemplating
>>> changing that
>>> though and using less machines.
>> Ah yes, I think that would indeed be desirable.
>
> I'll look into it, though it's not that trivial, since you might have a
>> So far I've set the execmask to 1 active channel, I'm contemplating
>> changing that
>> though and using less machines.
> Ah yes, I think that would indeed be desirable.
I'll look into it, though it's not that trivial, since you might have a 1x20x1
layout, also having to make sure each thread g
v2: limit lowerings to return statments in main()
Return statements in conditional blocks were not having their
output varyings lowered correctly.
This patch fixes the following piglit tests:
/spec/glsl-1.10/execution/vs-float-main-return
/spec/glsl-1.10/execution/vs-vec2-main-return
/spec/glsl-1
My intention was to have the render ring prelude occur at the first
point where we start emiting render commands into a batch.
Gen4-5 have a single ring that handles both BLT and 3D commands, so it's
possible to have a BLT -> RENDER transition in the middle of a batch
(not just at the start when p
Am 26.04.2016 um 23:18 schrieb Dave Airlie:
> On 27 April 2016 at 06:07, Roland Scheidegger wrote:
>> Am 26.04.2016 um 06:42 schrieb Dave Airlie:
>>> From: Dave Airlie
>>>
>>> This enables ARB_compute_shader on softpipe. I've only
>>> tested this with piglit so far, and I hopefully plan
>>> on in
From: Nanley Chery
Fixes some failures in dEQP-VK.api.info.image_format_properties.* and
enables the test group to execute without assert failing.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94896
Signed-off-by: Nanley Chery
---
src/intel/vulkan/anv_formats.c | 2 +-
1 file changed,
Quoting Jason Ekstrand (2016-04-26 14:31:10)
>
>
> On Tue, Apr 26, 2016 at 12:47 PM, Dylan Baker wrote:
>
> Quoting Jason Ekstrand (2016-04-25 21:39:19)
> > Previously, if an exception was encountered anywhere, nir_algebraic
> would
> > just die in a fire with no indication whatsoe
From: Dave Airlie
The GL4.1 spec bumps this to 2048, so we should do so.
Signed-off-by: Dave Airlie
---
src/gallium/drivers/softpipe/sp_limits.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/softpipe/sp_limits.h
b/src/gallium/drivers/softpipe/sp_limit
From: Dave Airlie
This is part of OES_shader_image_atomic.txt.
Signed-off-by: Dave Airlie
---
src/gallium/drivers/softpipe/sp_image.c | 40 +
1 file changed, 40 insertions(+)
diff --git a/src/gallium/drivers/softpipe/sp_image.c
b/src/gallium/drivers/softpipe/s
Reviewed-by: Bruce Cherniak , with addition of
.gitignore on commit.
-Original Message-
From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of Tim
Rowley
Sent: Tuesday, April 26, 2016 11:55 AM
To: mesa-dev@lists.freedesktop.org
Subject: [Mesa-dev] [PATCH] swr: autoge
https://bugs.freedesktop.org/show_bug.cgi?id=95159
--- Comment #1 from Youry ---
Created attachment 123283
--> https://bugs.freedesktop.org/attachment.cgi?id=123283&action=edit
Patch
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug._
https://bugs.freedesktop.org/show_bug.cgi?id=95159
Bug ID: 95159
Summary: Cannot build EGL without x11 headers after interop
patchset
Product: Mesa
Version: git
Hardware: ARM
OS: Linux (All)
Stat
On Tue, Apr 26, 2016 at 12:47 PM, Dylan Baker
wrote:
> Quoting Jason Ekstrand (2016-04-25 21:39:19)
> > Previously, if an exception was encountered anywhere, nir_algebraic would
> > just die in a fire with no indication whatsoever as to where the actual
> bug
> > is. This commit makes it print o
On Tue, Apr 26, 2016 at 12:27 PM, Jason Ekstrand
wrote:
> Status update:
>
> I've rebased on master, applied my own review comments and couple of bugs
> and pushed it here:
>
>
> https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/nir-foreach-block-v3
>
> Changes over what Connor sent:
> 1)
On 27 April 2016 at 06:07, Roland Scheidegger wrote:
> Am 26.04.2016 um 06:42 schrieb Dave Airlie:
>> From: Dave Airlie
>>
>> This enables ARB_compute_shader on softpipe. I've only
>> tested this with piglit so far, and I hopefully plan
>> on integrating it with my vulkan work. I'll get to
>> tes
Series is:
Reviewed-by: Chris Forbes
On Wed, Apr 27, 2016 at 3:33 AM, Thomas Helland
wrote:
> I guess you should also update GL4.4 section in GL3.txt.
> And add the extension to the release notes.
> Either a follow up patch or squashed into this one is fine with me.
>
> Regards,
> Thomas
>
> O
On Sun 17 Apr 2016, Laurent Carlier wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=94969
> ---
> src/intel/vulkan/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks for the fix. I pushed this to master with my r-b.
___
mes
On Sun 17 Apr 2016, Eduardo Lima Mitev wrote:
> Both patches are:
>
> Reviewed-by: Eduardo Lima Mitev
>
> Thanks!
> Eduardo
>
> On 04/16/2016 09:26 PM, Juha-Pekka Heikkila wrote:
> > These are just fixes for error paths.
> >
> > Juha-Pekka Heikkila (2):
> > meta: Avoid random memory access o
On Sat 16 Apr 2016, Grazvydas Ignotas wrote:
> For ctags users like me.
>
> Signed-off-by: Grazvydas Ignotas
> ---
> I have no commit access, if this patch is ok, please someone push.
Hi Grazvydas, I just pushed your patch.
___
mesa-dev mailing list
me
OpenGL 4.5 Core Profile section 7.1, in the documentation for
CompileShader, says: "Changing the source code of a shader object with
ShaderSource does not change its compile status or the compiled shader
code." (I haven't checked older versions of the spec.)
This test creates a shader, compiles it
Am 26.04.2016 um 06:42 schrieb Dave Airlie:
> From: Dave Airlie
>
> This enables ARB_compute_shader on softpipe. I've only
> tested this with piglit so far, and I hopefully plan
> on integrating it with my vulkan work. I'll get to
> testing it with deqp more later.
>
> The basic premise is to cr
Quoting Jason Ekstrand (2016-04-25 21:39:19)
> Previously, if an exception was encountered anywhere, nir_algebraic would
> just die in a fire with no indication whatsoever as to where the actual bug
> is. This commit makes it print out the particular search-and-replace
> expression that is causing
Hi Bo,
I am not aware of any environment variables that do that. However you
can change the limit in the source code.
You can change the single allocation size limit in
src/gallium/drivers/radeon/r600_pipe_common.c where max_mem_alloc_size
is set to 256MiB. You can change that, and then the total
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Marek Olšák changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Hi all,
First time poster here. Is there any way I can change global memory size
and max allocable memory size for an OpenCL program? My card has 4GB or
VRAM, but seems like I can only use a quarter of it for OCL, and I can only
use a quarter of the quarter for a single block of allocation. Is the
Status update:
I've rebased on master, applied my own review comments and couple of bugs
and pushed it here:
https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/nir-foreach-block-v3
Changes over what Connor sent:
1) Minor cosmetic changes to the iteration helper functions
2) Fixed a bug in
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Jose Fonseca changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |mar...@gmail.com
|org
On 04/26/2016 12:55 PM, Jose Fonseca wrote:
Ran `make check` inside src/glx to verify everything compiles and links
correctly.
https://bugs.freedesktop.org/show_bug.cgi?id=95158
---
src/glx/dri2_priv.h | 8
src/glx/glx_error.h
Ran `make check` inside src/glx to verify everything compiles and links
correctly.
https://bugs.freedesktop.org/show_bug.cgi?id=95158
---
src/glx/dri2_priv.h | 8
src/glx/glx_error.h | 8
src/glx/glxclient
https://bugs.freedesktop.org/show_bug.cgi?id=94955
--- Comment #22 from Roland Scheidegger ---
(In reply to David Lonie from comment #21)
> > I wonder what the test expects, I haven't seen anything but NaN coords with
> > that sampler ;-). That can't have possibly given really useful results,
>
On 04/26/2016 07:19 PM, Ilia Mirkin wrote:
Presumably this should also be CC'd to stable? I don't know when we'd
emit something like that, but presumably it can happen.
Well, it's only used by the lowering pass for surfaces, so I'm not sure
if it's really required to backport it because surf
Presumably this should also be CC'd to stable? I don't know when we'd
emit something like that, but presumably it can happen.
Does the GM107 emitter handle this case?
In any case, this series is
Reviewed-by: Ilia Mirkin
On Tue, Apr 26, 2016 at 1:11 PM, Samuel Pitoiset
wrote:
> This is pretty
https://bugs.freedesktop.org/show_bug.cgi?id=94994
Brian Paul changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
The third source must be emitted at offset 49 instead of 17 and the
not modifier is at 52 instead of 20. If you look a bit above in
emitLogicOp() you will see that the dest is emitted at 17 which
confirms that src(2) is obviously wrong.
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
src/gal
This is pretty similar to NVC0 except that offsets have changed.
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 26 ++
1 file changed, 26 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
b/src/gal
Could I put in a good word for adding it to .gitignore as well?
On Tue, Apr 26, 2016 at 12:55 PM, Tim Rowley wrote:
> ---
> src/gallium/drivers/swr/Makefile.am| 7 ++
> src/gallium/drivers/swr/Makefile.sources | 1 -
> .../rasterizer/jitter/scripts/gen_llvm_types.py
Reviewed-by: Ilia Mirkin
On Tue, Apr 26, 2016 at 12:52 PM, Samuel Pitoiset
wrote:
> This is loosely based on the previous lowering pass wrote by calim
> four years ago. I did clean the code and fixed some issues.
>
> v3: use getDst(0) for STORE indirect accesses
> v2: drop .raw code which is unu
Reviewed-by: Ilia Mirkin
On Tue, Apr 26, 2016 at 12:53 PM, Samuel Pitoiset
wrote:
> This fixes arb_shader_image_load_store-indexing and
> arb_shader_image_load_store-max-images.
>
> v2: - do not set r to 0xff that's just useless
> - use an assert to make sure tex.r == 0
> Signed-off-by: Samu
---
src/gallium/drivers/swr/Makefile.am| 7 ++
src/gallium/drivers/swr/Makefile.sources | 1 -
.../rasterizer/jitter/scripts/gen_llvm_types.py| 3 +
src/gallium/drivers/swr/swr_context.h | 8 +-
src/gallium/drivers/swr/swr_context_llvm.h |
This fixes arb_shader_image_load_store-indexing and
arb_shader_image_load_store-max-images.
v2: - do not set r to 0xff that's just useless
- use an assert to make sure tex.r == 0
Signed-off-by: Samuel Pitoiset
Reviewed-by: Ilia Mirkin (v1)
---
.../nouveau/codegen/nv50_ir_lowering_nvc0.cpp
This is loosely based on the previous lowering pass wrote by calim
four years ago. I did clean the code and fixed some issues.
v3: use getDst(0) for STORE indirect accesses
v2: drop .raw code which is unused for now
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_from_tgs
On 04/26/2016 03:22 AM, Ilia Mirkin wrote:
On Mon, Apr 25, 2016 at 4:15 PM, Samuel Pitoiset
wrote:
This fixes arb_shader_image_load_store-indexing and
arb_shader_image_load_store-max-images.
Signed-off-by: Samuel Pitoiset
Reviewed-by: Ilia Mirkin
---
.../nouveau/codegen/nv50_ir_lowering_n
Hi,
Any chance to get this looked at?
Thanks!
Eduardo
On 04/14/2016 06:52 PM, Eduardo Lima Mitev wrote:
> Hi,
>
> This is a series adding a new NIR pass that will combine redundant SSBO,
> shared variable and image load instructions. It is based on a previous series
> that Iago Toral [1] sent
On 04/26/2016 03:28 AM, Ilia Mirkin wrote:
On Mon, Apr 25, 2016 at 4:15 PM, Samuel Pitoiset
wrote:
3D images are a bit more complicated to implement and will probably
requires a bunch of headaches and we don't care for now because they
do not seem to be really used by apps.
v2: add a pipe_de
On 04/26/2016 03:17 AM, Ilia Mirkin wrote:
On Mon, Apr 25, 2016 at 4:15 PM, Samuel Pitoiset
wrote:
This implements RESQ for surfaces which comes from imageSize() GLSL
bultin. As the dimensions are sticked into the driver constant buffer,
this only has to be lowered with loads.
v2: - load a 1
On 04/26/2016 03:13 AM, Ilia Mirkin wrote:
On Mon, Apr 25, 2016 at 4:14 PM, Samuel Pitoiset
wrote:
This is loosely based on the previous lowering pass wrote by calim
four years ago. I did clean the code and fixed some issues.
v2: drop .raw code which is unused for now
Signed-off-by: Samuel
On 04/26/2016 03:08 AM, Ilia Mirkin wrote:
On Mon, Apr 25, 2016 at 4:14 PM, Samuel Pitoiset
wrote:
Old surfaces validation code will be removed once images are completely
done for Fermi/Kepler, that explains why I only disable it for now.
This also introduces nvc0_get_surface_dims() which co
I guess you should also update GL4.4 section in GL3.txt.
And add the extension to the release notes.
Either a follow up patch or squashed into this one is fine with me.
Regards,
Thomas
On Apr 26, 2016 12:25, "Kenneth Graunke" wrote:
>
> Stencil texturing is required by ES 3.1. Apparently we nev
From: Nicolai Hähnle
In optimized builds, visit(ir_expression *) experiences inlining with gcc that
leads the function to have a roughly 32KB stack frame. This is a problem given
that the function is called recursively. In non-optimized builds, the stack
frame is much smaller, hence one gets cras
On 04/25/2016 05:49 PM, Dave Airlie wrote:
From: Dave Airlie
For compute support some of the system values are .xyz types,
so move to using a vector instead of a single channel.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/draw/draw_gs.c | 2 +-
src/gallium/auxiliary/draw/draw
On 04/25/2016 09:06 PM, Dave Airlie wrote:
From: Dave Airlie
a) SysSemanticToIndex needs to be indexed with the semantic name
not the decl->Declaration.Semantic.
b) doing this in run is too late, as the mappings are all setup
prior to run in the execs.
This fixes
point-vertex-id gl_vertex_id
On 04/25/2016 10:42 PM, Dave Airlie wrote:
From: Dave Airlie
This lets us restart the machine at a PC value, and exits
the machine when we hit a barrier.
Compute shaders will then execute all the threads up to the
barrier, then restart the machines after the barrier once
all are done.
Signed-
On 04/26/2016 04:25 AM, Kenneth Graunke wrote:
Fixes
- ES31-CTS.gtf.GL31Tests.texture_stencil8.texture_stencil8
- ES31-CTS.gtf.GL31Tests.texture_stencil8.texture_stencil8_multisample
Signed-off-by: Kenneth Graunke
---
src/mesa/main/teximage.c | 9 +
1 file changed, 9 insertions(+)
d
https://bugs.freedesktop.org/show_bug.cgi?id=94955
--- Comment #21 from David Lonie ---
Thanks for the fix!
> Comment # 20 on bug 94955 from Roland Scheidegger
> (In reply to Bruce Cherniak from comment #19)
> > Thanks Roland! You patch fixes the segv I get when running both llvmpipe
> > and O
I have done some more debugging and I do have a question. The clearest,
repeatable issue I am seeing is when I run glxgears in apitrace. The crash
always occrs at GLXChooseVisual. It returns NULL and from the documentation
this means that it is encountering an undefined attribute. I see this when
I
Patches 1-5 are:
Reviewed-by: Iago Toral Quiroga
I'll have a look at patch 6 tomorrow.
On Mon, 2016-04-25 at 21:39 -0700, Jason Ekstrand wrote:
> This little series makes some bit size improvements to nir_algebraic. In
> particular, it adds a mechanism that allows you to specify a specific bit
On 04/22/2016 07:18 AM, Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/Makefile.sources | 1 +
> src/mesa/drivers/dri/i965/brw_context.c | 4 +-
> src/mesa/drivers/dri/i965/brw_context.h | 5 +
> src/mesa/drivers/dri/i965/brw_queryobj.c
On Tue, Apr 26, 2016 at 6:48 AM, Oded Gabbay wrote:
> Signed-off-by: Oded Gabbay
> Cc: "11.1 11.2"
Reviewed-by: Alex Deucher
> ---
> src/gallium/drivers/radeonsi/si_state.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_state.c
> b/s
It silence by default warnings with function parameters, as the
parameters need to be processed in order to have the actual and the
formal parameter, and the function signature. Then it raises the
warning if needed at verify_parameter_modes where other in/inout modes
checks are done.
v2: fix comme
Just to allow to call set_is_lhs on any ast_node without a casting. Useful
when processing a ast_node list that we know it contain ast_expression.
v2: comment out new_value to avoid unused parameter warning (Ian Romanick)
---
src/compiler/glsl/ast.h | 2 ++
src/compiler/glsl/ast_to_hir.c
On Tue, 2016-04-26 at 13:49 +0200, Ian Romanick wrote:
> On 04/26/2016 12:57 PM, Timothy Arceri wrote:
> >
> > On Tue, 2016-04-26 at 09:29 +0200, Ian Romanick wrote:
> > >
> > > On 04/26/2016 07:06 AM, Jamey Sharp wrote:
> > > >
> > > >
> > > > OpenGL 4.5 Core Profile section 7.1, in the docume
On 26/04/16 14:00, Ian Romanick wrote:
> Other than the formatting nits in patch 2, I think this series is fine.
Thanks for the feedback. I will update the patches and send a v2 soon.
> One nagging problem... we don't have any way to test this, and, as a
> result, none of it is tested. Can we f
Other than the formatting nits in patch 2, I think this series is fine.
One nagging problem... we don't have any way to test this, and, as a
result, none of it is tested. Can we fix that? There are 'make check'
tests in the Mesa tree for the preprocessor. Could we hook up something
like that to
On 04/19/2016 07:48 PM, Alejandro Piñeiro wrote:
> It silence by default warnings with function parameters, as the
> parameters need to be processed in order to have the actual and the
> formal parameter, and the function signature. Then it raises the
> warning if needed at verify_parameter_modes w
On 04/19/2016 07:48 PM, Alejandro Piñeiro wrote:
> Just to allow to call set_is_lhs on any ast_node without a casting. Useful
> when processing a ast_node list that we know it contain ast_expression.
> ---
>
> Worth to note that ast_node has other dummy implementations, like
> ast_node::hir.
>
>
On 04/26/2016 12:57 PM, Timothy Arceri wrote:
> On Tue, 2016-04-26 at 09:29 +0200, Ian Romanick wrote:
>> On 04/26/2016 07:06 AM, Jamey Sharp wrote:
>>>
>>> OpenGL 4.5 Core Profile section 7.1, in the documentation for
>>> CompileShader, says: "Changing the source code of a shader object
>>> with
>
Series is
Reviewed-by: Ian Romanick
The hack you remove in patch 4 always bothered me. Good riddance!
On 04/26/2016 12:25 PM, Kenneth Graunke wrote:
> We called intel_miptree_get_image_offset() to get the image offsets
> for the current level/slice, but then proceeded to ignore the results
> a
Gentle after-one-week ping.
On 19/04/16 19:48, Alejandro Piñeiro wrote:
> This series solves the false positive that Ilia found recently.
>
> On the thread I mention that my preferred option would be try to set
> is_lhs before processing the function parameters on a pass similar to
> verify_parame
Oded Gabbay wrote:
On Tue, Apr 26, 2016 at 1:26 PM, Andy Furniss wrote:
Oded Gabbay wrote:
Oded Gabbay (4): r600g/radeonsi: send endian info to format
translation functions r600g: set endianess of 16/32-bit buffers
according to do_endian_swap r600g: use do_endian_swap in color
swapping functi
On Tue, 2016-04-26 at 09:29 +0200, Ian Romanick wrote:
> On 04/26/2016 07:06 AM, Jamey Sharp wrote:
> >
> > OpenGL 4.5 Core Profile section 7.1, in the documentation for
> > CompileShader, says: "Changing the source code of a shader object
> > with
> > ShaderSource does not change its compile stat
On Tue, Apr 26, 2016 at 1:26 PM, Andy Furniss wrote:
> Oded Gabbay wrote:
>
>> Oded Gabbay (4): r600g/radeonsi: send endian info to format
>> translation functions r600g: set endianess of 16/32-bit buffers
>> according to do_endian_swap r600g: use do_endian_swap in color
>> swapping functions r600
Signed-off-by: Oded Gabbay
Cc: "11.1 11.2"
---
src/gallium/drivers/radeonsi/si_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index ae87a59..6017383 100644
--- a/src/gallium/drivers/ra
> From: Matt Turner
> Date: Mon, 25 Apr 2016 14:37:56 -0700
>
> On Tue, Apr 19, 2016 at 10:48 AM, Emil Velikov
> wrote:
> > On 19 April 2016 at 03:29, Jonathan Gray wrote:
> >> For systems without udev or sysfs that use drm ioctls in the loader
> >> drm authentication must take place earlier o
Oded Gabbay wrote:
Oded Gabbay (4): r600g/radeonsi: send endian info to format
translation functions r600g: set endianess of 16/32-bit buffers
according to do_endian_swap r600g: use do_endian_swap in color
swapping functions r600g: use do_endian_swap in texture swapping
function
I get a build
Stencil texturing is required by ES 3.1. Apparently we never actually
turned it on. Do that now. Also turn on the desktop extension.
Fixes nine dEQP-GLES31.functional tests:
stencil_texturing.format.stencil_index8_2d
texture.border_clamp.formats.stencil_index8.nearest_size_pot
texture.border_c
ES prohibits this, but GL appears to allow it. We at least need this
much, or else we'll crash as there's no source to read from.
This fixed crashes in the ES tests before I realized I needed to
prohibit stencil instead.
Signed-off-by: Kenneth Graunke
---
src/mesa/main/teximage.c | 5 +++--
1
We called intel_miptree_get_image_offset() to get the image offsets
for the current level/slice, but then proceeded to ignore the results
and clobber level/slice 0 every time.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94713
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i96
Fixes
- ES31-CTS.gtf.GL31Tests.texture_stencil8.texture_stencil8
- ES31-CTS.gtf.GL31Tests.texture_stencil8.texture_stencil8_multisample
Signed-off-by: Kenneth Graunke
---
src/mesa/main/teximage.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/mesa/main/teximage.c b/src/mesa/mai
Reviewed-by: Samuel Iglesias Gonsálvez
On 26/04/16 06:39, Jason Ekstrand wrote:
> This is consistent with the rename done for the rest of NIR. Currently,
> "bool" is the only type specifier used in nir_opt_algebraic.py so this is
> really a no-op.
> ---
> src/compiler/nir/nir_algebraic.py | 4 +
Reviewed-by: Samuel Iglesias Gonsálvez
On 26/04/16 06:39, Jason Ekstrand wrote:
> ---
> src/compiler/nir/nir_opt_algebraic.py | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/compiler/nir/nir_opt_algebraic.py
> b/src/compiler/nir/nir_opt_algebraic.py
> index f
1 - 100 of 106 matches
Mail list logo
