Am 19.07.19 um 18:18 schrieb Ilia Mirkin:
On Fri, Jul 19, 2019 at 12:07 PM Tobias Klausmann
wrote:
On 19.07.19 15:39, Eric Engestrom wrote:
On Friday, 2019-07-19 13:56:30 +0200, Mark Menzynski wrote:
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=111007
Fixes: https
On 19.07.19 15:39, Eric Engestrom wrote:
On Friday, 2019-07-19 13:56:30 +0200, Mark Menzynski wrote:
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=111007
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=67
`Fixes:` is used to indicate the commit that introduced the code being
fixe
src/vulkan/meson.build as they apply to the overlay, utils & wsi.
Since it's correct though, I landed Tobias' patch with my r-b and
Cc: stable (I dropped the unnecessary [] though), and I sent MR !460
with your suggestion.
Thanks,
-Lionel
On 16/03/2019 18:56, Tobias Klausmann w
terminated.
The above misses the include directory for wayland:
-I/usr/include/wayland
Signed-off-by: Tobias Klausmann
---
src/vulkan/util/meson.build | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/src/vulkan/util/meson.build b/src/vulkan/util/meson.build
index
On 17.12.18 23:16, Dylan Baker wrote:
Quoting Tobias Klausmann (2018-12-17 14:05:13)
On 17.12.18 20:33, Dylan Baker wrote:
Quoting Emil Velikov (2018-12-17 11:10:00)
On Fri, 14 Dec 2018 at 17:57, Dylan Baker wrote:
Quoting Emil Velikov (2018-12-13 08:15:57)
From: Emil Velikov
Reasonably
On 17.12.18 20:33, Dylan Baker wrote:
Quoting Emil Velikov (2018-12-17 11:10:00)
On Fri, 14 Dec 2018 at 17:57, Dylan Baker wrote:
Quoting Emil Velikov (2018-12-13 08:15:57)
From: Emil Velikov
Reasonably often people will want to build the loader w/o any drivers.
Be that debugging an issue
yland:
-I/usr/include/wayland
The missing include is contained in the (until now) unused radv_deps:
if with_platform_wayland
radv_deps += dep_wayland_client
radv_flags += '-DVK_USE_PLATFORM_WAYLAND_KHR'
libradv_files += files('radv_wsi_wayland.c')
endif
Fixes:
Hi,
fyi: there is another bugreport #106372 [1], where i bisected the
problem in the xserver and found a problematic commit, with code which
can easily be reverted (patch in the bugreport), maybe you could check
if that fixes the issue as well!
PS: I looked into bugzilla last weekend where i
w_reg*,
const brw_reg*)':
compiler/brw_reg.h:305:1: warning: control reaches end of non-void function
[-Wreturn-type]
Signed-off-by: Tobias Klausmann
---
src/intel/compiler/brw_reg.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/intel/compiler/brw_reg.h b/src/intel/compiler/brw_r
Hi,
comments down below...
Greetings,
Tobias
On 12/2/17 12:34 PM, Karol Herbst wrote:
From: Karol Herbst
Creating correct SubOps for OP_MADSP wasn't easy, because devs needed to know
the proper values for each data type. Also the third source doesn't know any
explicit signess and the curr
On 11/12/17 4:16 PM, Ilia Mirkin wrote:
On Sun, Nov 12, 2017 at 9:09 AM, Tobias Klausmann
wrote:
On 11/12/17 3:53 AM, Ilia Mirkin wrote:
It's common to use signed int modulo in GLSL. As it happens, the GLSL
specs allow the result to be undefined, but that seems fairly
surprising. It&
og, i);
}
break;
lgtm,
Reviewed-by: Tobias Klausmann
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 11/12/17 3:53 AM, Ilia Mirkin wrote:
It's common to use signed int modulo in GLSL. As it happens, the GLSL
specs allow the result to be undefined, but that seems fairly
surprising. It's not that much more effort to get it right, at least for
positive modulo operators.
Signed-off-by: Ilia Mir
renamed in envytools
and not synced yet?). A bit off topic question is: Where does nvc0 set
those regs? I have a hard time finding it. :O
Anyway, the patch looks ok and you can add my R-b with or without
renaming the regs:
Reviewed-by: Tobias Klausmann
/after shader-db run would be nice to see.
Reviewed-by: Tobias Klausmann
>
> Signed-off-by: Ilia Mirkin
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 2 ++
> src/gallium/drivers/nouveau/nv50/nv50_screen.c| 2 +-
> src/gallium/drivers/nou
ackend.h
Oh,
yep this is better! I should have changed it when fixing the build a few
weeks ago.
Acked-by: Tobias Klausmann
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 10/4/17 3:47 PM, Emil Velikov wrote:
> On 3 October 2017 at 14:45, Tobias Klausmann
> wrote:
>> Starting with commit ab0589c6ed ("wayland-egl: remove no longer needed
>> wayland-client dependency") the wayland-egl.h include was missing leading to
>&
t;wayland-egl.h"
^~~
Fixes: ab0589c6ed ("wayland-egl: remove no longer needed wayland-client
dependency")
Signed-off-by: Tobias Klausmann
Reviewed-by: Emil Velikov
---
src/egl/wayland/wayland-egl/Makefile.am | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
On 10/3/17 3:37 PM, Emil Velikov wrote:
> On 3 October 2017 at 14:34, Tobias Klausmann
> wrote:
>> Starting with commit ab0589c6ed ("wayland-egl: remove no longer needed
>> wayland-client dependency") the wayland-egl.h include was missing leading to
>&
gl.h include!
Fixes: ab0589c6ed ("wayland-egl: remove no longer needed wayland-client
dependency")
Signed-off-by: Tobias Klausmann
Reviewed-by: Emil Velikov
---
src/egl/wayland/wayland-egl/wayland-egl-backend.h | 16
src/egl/wayland/wayland-egl/wayland-egl.c
e "wayland-egl.h"
^~~~~~~
Signed-off-by: Tobias Klausmann
Fixes: ab0589c6ed ("swayland-egl: remove no longer needed wayland-client
dependency")
---
src/egl/wayland/wayland-egl/Makefile.am | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
Wether we wait on an inverted rendering condition or not, we should not render
on a passed query.
This fixes the CTS test case 'KHR-GL45.conditional_render_inverted.functional'.
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 2 +-
1 file
0 0 0
Signed-off-by: Tobias Klausmann
---
.../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_lowering_nvc0.cpp
b/src/gallium/drivers/n
When building sandboxed, we may encounter additional errors. Ignore the errors,
as we are in a constrained environment.
This can be observed when building latest git with OBS.
Signed-off-by: Tobias Klausmann
---
bin/git_sha1_gen.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/bin
This lets us spill more values and compile a big shader for Civilization 6.
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp
b/src/gallium/drivers
Looks good to me!
Reviewed-by: Tobias Klausmann
On 7/30/17 5:51 PM, Karol Herbst wrote:
> For mul(a, +-1) codegen could end up generating OP_MOV with a saturation flag
> set which we got simply ignored.
>
> Adding an assert for detecting more of such issues.
>
> Fixes wrongly
Tested-by: Tobias Klausmann
On 6/12/17 1:47 AM, Dave Airlie wrote:
From: Dave Airlie
The max_array_access field applies to the first dimension, which means
we only want to set it for the 1D clip dist arrays.
This fixes an ir_validate assert seen with
KHR-GL44.cull_distance.functional
on
t;builder, src0,
ctx->i32one, ""));
^~~~~~~
On Wed, 1 Mar 2017 at 21:22 Tobias Klausmann
<mailto:tobias.johannes.klausm...@mni.thm.de>> wrote:
Fix usage of ac_add_function_attr() and make
main_function, i + 1, AC_FUNC_ATTR_BYVAL);
^~~~
Signed-off-by: Tobias Klausmann
---
src/amd/common/ac_llvm_util.c | 3 +--
src/amd/common/ac_llvm_util.h | 2 ++
src/amd/common/ac_nir_to_llvm.c | 4 ++--
3 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/amd/c
On 02.10.2016 13:56, Nicolai Hähnle wrote:
On 01.10.2016 22:22, Tobias Klausmann wrote:
On 01.10.2016 21:46, Marek Olšák wrote:
Hi,
I propose that we use versioning in the form of "year.quarter".
2017 would start with 17.0, then 17.1, 17.2, 17.3 for following
quarters o
On 01.10.2016 21:46, Marek Olšák wrote:
Hi,
I propose that we use versioning in the form of "year.quarter".
2017 would start with 17.0, then 17.1, 17.2, 17.3 for following
quarters of the year, respectively.
2018 would start with 18.0, then 18.1, 18.2, 18.3.
The motivation is that you can ea
This was found by obs:
I: Program returns random data in a function
E: Mesa no-return-in-nonvoid-function main/program_resource.c:109
Signed-off-by: Tobias Klausmann
Reviewed-by: Ian Romanick
---
V2:
Remove the !, cosmetic change, as the macro expansion stringifies everything:
__assert_fail
This was found by obs:
I: Program returns random data in a function
E: Mesa no-return-in-nonvoid-function main/program_resource.c:109
Signed-off-by: Tobias Klausmann
---
src/mesa/main/program_resource.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main
ral in update_framebuffer_size().
Greetings,
Tobias
[1] https://homepages.thm.de/~tjkl80/openmw_backtrace.txt
On 18.08.2016 11:55, Marek Olšák wrote:
Hi,
Do you have a test case?
Marek
On Thu, Aug 18, 2016 at 1:44 AM, Tobias Klausmann
<mailto:tobias.johannes.klausm...@mni.thm.de>> wrot
OpenMW tries to upload a new surface (mouse pointer) which fails in the now
guarded update_framebuffer_size() as the surface is NULL.
This is not inteded as a real "fix", as it would just hide the immediate crash.
So if somebody could take a look at this...
Reported-by:
Signed-off-
Hi Karol,
one question inline.
On 13.08.2016 12:02, Karol Herbst wrote:
min/max pairs can be dual issued on Kepler1
changes for ./GpuTest /test=pixmark_piano /benchmark /no_scorebox /msaa=0
/benchmark_duration_ms=6 /width=1024 /height=640:
without dual_issue pass:
inst_executed: 1.03G
in
== OPCLASS_FLOW)
return false;
+
+ if (b->dependsOn(a))
+ return false;
+
// anything with MOV
if (a->op == OP_MOV || b->op == OP_MOV)
return true;
Reviewed-by: Tobias Klausmann
___
mesa-dev
;
+ virtual bool hasDualIssueing() const;
virtual bool canDualIssue(const Instruction *, const Instruction *) const;
virtual int getLatency(const Instruction *) const;
virtual int getThroughput(const Instruction *) const;
Reviewed-by: Tobias Klausmann
___
On 04.06.2016 01:22, Emil Velikov wrote:
From: Emil Velikov
Otherwise we might fail to find the headers in some scenarios.
Cc: Tobias Klausmann
Reported-by: Tobias Klausmann
Signed-off-by: Emil Velikov
---
src/intel/vulkan/Makefile.am | 2 ++
1 file changed, 2 insertions(+)
diff --git
On 04.06.2016 00:54, Emil Velikov wrote:
On 3 June 2016 at 23:03, Tobias Klausmann
wrote:
With header in uncommen (non-system findable places) the build fails, use
the already found places to include those headers.
Are you sure about this ? What exactly is your build setup/config
(in-tree
With header in uncommen (non-system findable places) the build fails, use
the already found places to include those headers.
Signed-off-by: Tobias Klausmann
---
src/intel/vulkan/Makefile.am | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/Makefile.am b
On 02.06.2016 23:54, Rob Herring wrote:
On Thu, Jun 2, 2016 at 4:45 PM, Tobias Klausmann
wrote:
Starting with commit e8b38ca202fbe8c281aeb81a4b64256983f185e0 the dummy_thread
misses the proper intialization for the new member, leading to the error stated
at the bottom. This patch initializes
Starting with commit e8b38ca202fbe8c281aeb81a4b64256983f185e0 the dummy_thread
misses the proper intialization for the new member, leading to the error stated
at the bottom. This patch initializes the new member.
Signed-off-by: Tobias Klausmann
CC main/eglcurrent.lo
In file included
On 01.06.2016 19:31, Eric Engestrom wrote:
On Wed, Jun 01, 2016 at 12:18:57AM +0200, Tobias Klausmann wrote:
Signed-off-by: Tobias Klausmann
---
src/Makefile.am | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/Makefile.am b/src/Makefile.am
index f5c0773..d0990dc
Signed-off-by: Tobias Klausmann
---
src/Makefile.am | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/Makefile.am b/src/Makefile.am
index f5c0773..d0990dc 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -22,8 +22,8 @@
git_sha1.h:
@if test -e $(top_srcdir
Reviewed-by: Tobias Klausmann
On 25.05.2016 01:59, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
This addresses the feedback I got after pushing the enablement patch.
docs/relnotes/11.3.0.html | 1 +
src/compiler/glsl/builtin_variables.cpp | 10 --
src
On 24.05.2016 03:42, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
src/compiler/glsl/builtin_variables.cpp | 12 +++
src/compiler/glsl/glsl_parser_extras.cpp | 1 +
src/compiler/glsl/glsl_parser_extras.h | 2 ++
src/compiler/glsl/linker.cpp | 35 ++
Latest master head compiles fine here without this patch for a small
build (i965 and nouveau)
On 16.05.2016 15:52, Daniel Vetter wrote:
No idea why, but without this my mesa stopped compiling ...
Signed-off-by: Daniel Vetter
---
src/mesa/drivers/dri/nouveau/nouveau_screen.c | 1 +
1 file
Looks a bit silly to review this patch as it is run under my name, but
anyway:
Reviewed-by: Tobias Klausmann
On 14.05.2016 16:21, Ilia Mirkin wrote:
From: Tobias Klausmann
Cull distances are just a special case of clip distances as far as the
hardware is concerned. Make sure that the
Hi Dave,
i was not aware you are working on this actively as well, i had a series
posted 5 days ago which got some critics and reviews [1]. The most
important points where:
1. split functional change and renaming of the lowering pass [Ian]
2. check max clip/cull array sizes in link_shaders f
On 08.05.2016 23:29, Tobias Klausmann wrote:
On 08.05.2016 22:50, Ilia Mirkin wrote:
What exactly gets fed into the CLIPDIST and CULLDIST semantics? e.g.
is CULLDIST[0].x the first cull distance, or is it the first entity in
the combined cull/clip distance array? If the former, then this won
On 09.05.2016 20:41, Ian Romanick wrote:
snip
+ /* From the ARB_cull_distance spec:
+ *
+ * It is a compile-time or link-time error for the set of shaders forming
+ * a program to have the sum of the sizes of the gl_ClipDistance and
+ * gl_CullDistance arrays to be
mesa/commit/?id=5227e915803079e5e72a0b2fde3a11d62af8df99
Greetings,
Tobias
On Sun, May 8, 2016 at 4:44 PM, Tobias Klausmann
wrote:
After the cleanup of my patches in v2, this is another take on finishing this
extension.
v2: cleanup, reordering of patches, split lowering pass adapation (Dave Airlie)
v3:
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/llvmpipe/lp_screen.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c
b/src/gallium/drivers/llvmpipe/lp_screen.c
index c6c18ee..6aef5c9 100644
--- a/src/gallium/drivers/llvmpipe
Signed-off-by: Tobias Klausmann
---
src/mesa/state_tracker/st_extensions.c | 1 +
src/mesa/state_tracker/st_program.c| 40 ++
2 files changed, 41 insertions(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
This lets us safely enable or disable the extension as needed
Signed-off-by: Tobias Klausmann
Reviewed-by: Edward O'Callaghan
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/i915/i915_scr
From: Dave Airlie
This just renames the file in anticipation of adding cull lowering.
Signed-off-by: Tobias Klausmann
Signed-off-by: Dave Airlie
Reviewed-by: Edward O'Callaghan
---
src/compiler/Makefile.sources | 2 +-
src/compiler/glsl/lower_clip_distance.cpp
This enables ARB_cull_distance.
Signed-off-by: Tobias Klausmann
---
docs/GL3.txt| 2 +-
docs/relnotes/11.3.0.html | 1 +
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 2 ++
src/gallium/drivers/nouveau/nvc0/nvc0_program.h | 1 +
src
airlied:
v2: rename LowerClipDistance to LowerCombinedClipCullDistnace.
I don't think we want any other behaviour with any current hw.
Signed-off-by: Tobias Klausmann
Reviewed-by: Edward O'Callaghan
---
src/compiler/glsl/link_varyings.cpp | 2 +-
src/compiler/glsl/
This will come in handy when we want to lower gl_CullDistance into
gl_CullDistanceMESA.
[airlied: drop separate APIs for clip/cull - just use single API
to call both passes.]
Signed-off-by: Tobias Klausmann
---
src/compiler/glsl/ir_optimization.h | 3 +-
src/compiler/glsl/linker.cpp
Signed-off-by: Tobias Klausmann
---
src/compiler/glsl/ast_to_hir.cpp | 14
src/compiler/glsl/builtin_variables.cpp | 11 ++-
src/compiler/glsl/glcpp/glcpp-parse.y| 3 +
src/compiler/glsl/glsl_parser_extras.cpp | 1 +
src/compiler/glsl/glsl_parser_extras.h
Signed-off-by: Tobias Klausmann
Reviewed-by: Edward O'Callaghan
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
b/src/gallium/drivers/nouveau/co
Signed-off-by: Tobias Klausmann
Reviewed-by: Edward O'Callaghan
---
src/mapi/glapi/gen/gl_API.xml | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index 4bfa9ef..2fcba0b 100644
--- a/src/mapi/glap
if an array was unsized
- fix GL3.txt, add releasenote
Dave Airlie (1):
glsl: rename lower_clip_distance to lower_distance.
Tobias Klausmann (10):
glapi: Add GL_ARB_cull_distance
mesa/main: Add support for GL_ARB_cull_distance (v2)
mesa/prog: Add varyings for arb_cull_distance
glsl
Signed-off-by: Tobias Klausmann
Reviewed-by: Edward O'Callaghan
---
src/mesa/program/prog_print.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/program/prog_print.c b/src/mesa/program/prog_print.c
index 08381b4..755d644 100644
--- a/src/mesa/program/prog_print.c
+++
On 05.04.2016 01:32, Timothy Arceri wrote:
On Mon, 2016-04-04 at 15:07 +0200, Tobias Klausmann wrote:
On 04.04.2016 04:48, Timothy Arceri wrote:
On Mon, 2016-04-04 at 12:15 +1000, Dave Airlie wrote:
From: Tobias Klausmann
Signed-off-by: Tobias Klausmann Const.MaxClipPlanes);
+ } else if
On 05.04.2016 00:06, Dave Airlie wrote:
On 4 April 2016 at 23:07, Tobias Klausmann
wrote:
On 04.04.2016 04:48, Timothy Arceri wrote:
On Mon, 2016-04-04 at 12:15 +1000, Dave Airlie wrote:
From: Tobias Klausmann
Signed-off-by: Tobias Klausmann Const.MaxClipPlanes);
+ } else if (strcmp
On 04.04.2016 04:32, Ilia Mirkin wrote:
On Sun, Apr 3, 2016 at 10:15 PM, Dave Airlie wrote:
From: Tobias Klausmann
This enables ARB_cull_distance.
This isn't enough - something needs to lower the CULLDIST into
CLIPDIST (or some combined thing). Ideally they'd both just come in a
On 04.04.2016 04:48, Timothy Arceri wrote:
On Mon, 2016-04-04 at 12:15 +1000, Dave Airlie wrote:
From: Tobias Klausmann
Signed-off-by: Tobias Klausmann Const.MaxClipPlanes);
+ } else if (strcmp("gl_CullDistance", name) == 0
+ && size > state
er of uniform blocks allowed for
compute shaders is less than 12. This happens on Kepler but this might
also affect other Gallium drivers.
Signed-off-by: Samuel Pitoiset
Reported-by: Tobias Klausmann
Tested-by: Tobias Klausmann
---
src/mesa/state_tracker/st_extensions.c | 8
1 file chang
On 18.10.2015 19:14, Marek Olšák wrote:
From: Marek Olšák
This is more practical and needed by gallium.
---
src/glsl/linker.cpp| 33 +++--
src/glsl/nir/glsl_to_nir.cpp | 3 ++-
src/mesa/drivers/dri/i965/brw_vs.c | 2 +-
src/mesa/main/mt
On 18.10.2015 19:52, Marek Olšák wrote:
On Sun, Oct 18, 2015 at 7:25 PM, Ilia Mirkin wrote:
Patches 1-3 are Reviewed-by: Ilia Mirkin . Not
familiar with the tgsi scan stuff.
Do you have plans on finishing cull distance support btw? Dave Airlie
and Tobias Klaussman have both made attempts at
On 31.08.2015 18:23, Ilia Mirkin wrote:
On Mon, Aug 31, 2015 at 12:21 PM, Tobias Klausmann
wrote:
On 31.08.2015 17:19, Marcos Paulo de souza wrote:
ping :)
Em 27-08-2015 12:59, Marcos Paulo de Souza escreveu:
instead of call prog->getTarget(), since the target never change in prog.
On 31.08.2015 17:19, Marcos Paulo de souza wrote:
ping :)
Em 27-08-2015 12:59, Marcos Paulo de Souza escreveu:
instead of call prog->getTarget(), since the target never change in
prog.
It is not changing now, don't know if it will in the future. Using
prog->getTarget() may save us from has
On 24.08.2015 17:51, Ilia Mirkin wrote:
The hardware only generates vertexid when vertices come from a VBO. This
fixes:
vertexid-drawelements
vertexid-drawarrays
Signed-off-by: Ilia Mirkin
Cc: "11.0"
---
src/gallium/drivers/nouveau/nv50/nv50_program.c| 1 +
src/gallium/driv
This will come in handy when we want to lower gl_CullDistance into
gl_CullDistanceMESA.
Signed-off-by: Tobias Klausmann
---
src/glsl/Makefile.sources| 2 +-
src/glsl/ir_optimization.h | 1 +
src/glsl/lower_clip_distance.cpp | 574
src/glsl
Mh especially GCC 5 seems to be picky about code, if we don't work
around we soon will only see warnings for every file. Even if the
compiler is buggy/to picky we should fix the code to not drown in
warnings... :-)
On 05.06.2015 22:23, Ilia Mirkin wrote:
Sure. Fix it in GCC :) disable the bo
pong
the unhappyness case will hopefully soon be extended with cull_distance
+ clipvertex, so good to see this fixed!
Reviewed-by: Tobias Klausmann
On 25.05.2015 21:42, Ilia Mirkin wrote:
ping
On Sun, May 17, 2015 at 5:56 PM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
Just
least for llvmpipe, I'm not entirely sure for softpipe) if not
there's some unaccounted difference somewhere how we'd thought of how
this should work for dx10 vs. opengl).
Hey nice to know llvmpipe already supports this, i'll test it!
Tobias
Roland
Am 24.05.2015 um
Can you push this with your newly won commit access? Makes up a nice
trial ;-)
Am 25.05.2015 um 16:49 schrieb Samuel Pitoiset:
yeah, very important thing to do *before* starting to write a new
extension. ;)
Reviewed-by: Samuel Pitoiset
On 05/25/2015 03:57 PM, Tobias Klausmann wrote
Signed-off-by: Tobias Klausmann
---
I'm already getting emails wanting me to do this, so just mark it,
wont change anything really
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 44a824b..8e1c8cd 100644
--- a/docs/GL3.txt
On 25.05.2015 01:19, Timothy Arceri wrote:
On Mon, 2015-05-25 at 00:46 +0200, Tobias Klausmann wrote:
hi,
replay inline.
On 25.05.2015 00:34, Timothy Arceri wrote:
On Sun, 2015-05-24 at 19:58 +0200, Tobias Klausmann wrote:
Signed-off-by: Tobias Klausmann
---
src/glsl/ast_to_hir.cpp
hi,
replay inline.
On 25.05.2015 00:34, Timothy Arceri wrote:
On Sun, 2015-05-24 at 19:58 +0200, Tobias Klausmann wrote:
Signed-off-by: Tobias Klausmann
---
src/glsl/ast_to_hir.cpp | 14 +
src/glsl/builtin_variables.cpp | 13 +++-
src/glsl/glcpp/glcpp-parse.y
Signed-off-by: Tobias Klausmann
---
src/glsl/ast_to_hir.cpp | 14 +
src/glsl/builtin_variables.cpp | 13 +++-
src/glsl/glcpp/glcpp-parse.y| 3 +
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 3 +
src/glsl/link_varyings.cpp
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
index ecd115f..381a958
Signed-off-by: Tobias Klausmann
---
src/mesa/program/prog_print.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/program/prog_print.c b/src/mesa/program/prog_print.c
index d588d07..e8855cd 100644
--- a/src/mesa/program/prog_print.c
+++ b/src/mesa/program/prog_print.c
@@ -147,6
From: Dave Airlie
Signed-off-by: Tobias Klausmann
---
src/glsl/Makefile.sources| 1 +
src/glsl/ir_optimization.h | 1 +
src/glsl/link_varyings.cpp | 15 +-
src/glsl/link_varyings.h | 3 +-
src/glsl/linker.cpp | 1 +
src/glsl
Signed-off-by: Tobias Klausmann
---
src/mapi/glapi/gen/gl_API.xml | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index 3090b9f..a792056 100644
--- a/src/mapi/glapi/gen/gl_API.xml
+++ b/src/mapi/glapi/gen
Signed-off-by: Tobias Klausmann
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_gs.c | 2 +-
src/mesa/drivers/dri/i965/brw_vec4.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vs.c
Add another pipe cap so we can savely enable of disable this extension
Signed-off-by: Tobias Klausmann
---
src/gallium/auxiliary/cso_cache/cso_context.c| 3 +++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/i915/i915_screen.c | 1 +
src/gallium
Signed-off-by: Tobias Klausmann
---
src/mesa/state_tracker/st_extensions.c | 4
src/mesa/state_tracker/st_program.c| 34 ++
2 files changed, 38 insertions(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
This enables ARB_cull_distance.
Signed-off-by: Tobias Klausmann
---
docs/GL3.txt | 2 +-
docs/relnotes/10.7.0.html | 4 +++-
src/gallium/drivers/nouveau/nvc0/nvc0_program.c| 6 +-
src/gallium/drivers/nouveau/nvc0
This patch series adds the needed support for this extension to the various
parts of mesa to finally enable it for nvc0.
Dave Airlie (1):
glsl: lower cull_distance into cull_distance_mesa
Tobias Klausmann (10):
glapi: add GL_ARB_cull_distance
mesa/main: add support for GL_ARB_cull_distance
Signed-off-by: Tobias Klausmann
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/get.c | 26 ++
src/mesa/main/get_hash_params.py | 4
src/mesa/main/mtypes.h | 22 +-
src/mesa/main/shaderapi.c
Signed-off-by: Tobias Klausmann
---
src/glsl/glsl_types.cpp | 8
src/glsl/glsl_types.h | 10 --
src/glsl/linker.cpp | 2 +-
3 files changed, 13 insertions(+), 7 deletions(-)
diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cpp
index f675e90..4bc7324 100644
Reviewed-by: Tobias Klausmann
On 23.05.2015 18:56, Ilia Mirkin wrote:
There can be scenarios where the "indirect" arg of a PFETCH becomes
known, and so the code will attempt to propagate it. Use this
opportunity to just fold it into the first argument, and prevent the
load propag
Reviewed-by: Tobias Klausmann
On 18.05.2015 11:29, Marek Olšák wrote:
From: Marek Olšák
This fixes a crash in nouveau which can't handle
set_constant_buffer(PIPE_SHADER_TESS_*).
where is the S-o-b ? ;-)
---
src/gallium/auxiliary/cso_cache/cso_context.c | 7 +++
src
On 18.05.2015 10:43, Samuel Pitoiset wrote:
Thanks for the review.
Can I get your R-b for the other patches of the series. :-)
Oh silly me, it was meant to be a R-b for patch 1+2 on both series, sorry :)
On 05/17/2015 06:46 PM, Tobias Klausmann wrote:
Reviewed-by: Tobias Klausmann
On
Reviewed-by: Tobias Klausmann
On 17.05.2015 18:21, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/Makefile.sources| 1 +
src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 12 +--
src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 28
Reviewed-by: Tobias Klausmann
On 17.05.2015 18:19, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/Makefile.sources| 1 +
src/gallium/drivers/nouveau/nv50/nv50_context.h | 12 +---
src/gallium/drivers/nouveau/nv50/nv50_query.c | 21
1 - 100 of 191 matches
Mail list logo