It's just a simple, portable way to make the `unused parameter`
warning shut up, when you're deliberately ignoring your parameters.
There are other ways [__attribute__(( unused )), etc] but they cease
to be simple or portable fairly quickly.
-- Chris
On Mon, Aug 26, 2013 at 5:07 PM, Liu Xin
On Sun, Aug 25, 2013 at 10:07 PM, Liu Xin liu...@icubecorp.com wrote:
HI, list,
i don't understand why glsl source have so many 'cast to void' operations,
like this.
ir_rvalue *
ast_node::hir(exec_list *instructions,
struct _mesa_glsl_parse_state *state)
{
(void)
Previously, we allocated space in brw_vs_prog_data's params and
pull_params arrays for MAX_CLIP_PLANES vec4s---even when it wasn't
necessary.
On a 64-bit architecture, this used 0.5 kB of space (8 clip planes *
4 floats per plane * 8 bytes per float pointer * 2 arrays of pointers =
512 bytes).
Hi TGSI guys mostly :-)
So I'm wondering how circular and perfect tgsi-text-tgsi roundabouts
should be,
currently the TGSI dump code uses .4f in one place, which makes things
like 1e6 not make it across the divide, I was thinking of dumping
immediates in 32-bit hex format so we know for definite
On 08/26/2013 02:42 PM, Matt Turner wrote:
On Sun, Aug 25, 2013 at 10:07 PM, Liu Xin liu...@icubecorp.com wrote:
HI, list,
i don't understand why glsl source have so many 'cast to void' operations,
like this.
ir_rvalue *
ast_node::hir(exec_list *instructions,
struct
The following patchset implements the KHR_debug extension.
Here are some notes/assumptions I have made:
* These patches only enable the extension for GL not ES
* MAX_LABEL_LENGTH and MAX_DEBUG_GROUP_STACK_DEPTH are set to the minimum
values defined in the spec.
* Assume its fine that
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/config.h |7 ++-
src/mesa/main/errors.c |4
src/mesa/main/mtypes.h | 15 ++-
3 files changed, 20 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/config.h b/src/mesa/main/config.h
index
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/errors.c | 308 +---
src/mesa/main/errors.h | 15 +++
2 files changed, 226 insertions(+), 97 deletions(-)
diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c
index
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/hash.c | 26 ++
src/mesa/main/hash.h |3 +++
2 files changed, 29 insertions(+)
diff --git a/src/mesa/main/hash.c b/src/mesa/main/hash.c
index 6591af9..8dde8b1 100644
--- a/src/mesa/main/hash.c
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/errors.c | 275
src/mesa/main/errors.h |5 +
src/mesa/main/mtypes.h |5 +-
3 files changed, 214 insertions(+), 71 deletions(-)
diff --git a/src/mesa/main/errors.c
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/arrayobj.c | 10 +-
src/mesa/main/arrayobj.h |3 +++
2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/arrayobj.c b/src/mesa/main/arrayobj.c
index 9226050..006a2ea 100644
---
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/dlist.c | 12 ++--
src/mesa/main/dlist.h |2 ++
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index af2b468..a82436c 100644
---
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/mtypes.h | 12
1 file changed, 12 insertions(+)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index bd3560a..2277b95 100644
--- a/src/mesa/main/mtypes.h
+++ b/src/mesa/main/mtypes.h
@@ -1128,6
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/arrayobj.c |1 +
src/mesa/main/bufferobj.c|1 +
src/mesa/main/dlist.c|1 +
src/mesa/main/framebuffer.c |1 +
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/syncobj.c |2 +-
src/mesa/main/syncobj.h |3 +++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/syncobj.c b/src/mesa/main/syncobj.c
index e426474..92c7cb0 100644
---
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/objectlabel.c | 277 +++
src/mesa/main/objectlabel.h | 61 ++
2 files changed, 338 insertions(+)
create mode 100644 src/mesa/main/objectlabel.c
create mode 100644
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/Makefile.sources |1 +
src/mesa/SConscript |1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/Makefile.sources b/src/mesa/Makefile.sources
index a5c1f5d..122ea8e 100644
--- a/src/mesa/Makefile.sources
+++
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/drivers/dri/i965/brw_context.c |2 ++
src/mesa/main/enable.c |9 +
src/mesa/main/errors.c |3 +++
src/mesa/main/mtypes.h |1 +
Remap any type or severity exclusive to KHR_debug to
something suitable for ARB_debug_output
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/errors.c | 65 ++--
src/mesa/main/mtypes.h |1 +
2 files changed, 58 insertions(+),
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/extensions.c |3 +++
src/mesa/main/get_hash_params.py | 14 +-
2 files changed, 12 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mapi/glapi/gen/gl_API.xml| 149 +-
src/mapi/glapi/gen/gl_genexec.py |1 +
2 files changed, 149 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml
https://bugs.freedesktop.org/show_bug.cgi?id=67046
Michel Dänzer mic...@daenzer.net changed:
What|Removed |Added
CC||arek.r...@gmail.com
On 08/26/2013 02:38 AM, Dave Airlie wrote:
Hi TGSI guys mostly :-)
So I'm wondering how circular and perfect tgsi-text-tgsi roundabouts
should be,
Ideally, they should be consistent.
currently the TGSI dump code uses .4f in one place, which makes things
like 1e6 not make it across the
On 08/26/2013 04:43 AM, Timothy Arceri wrote:
The following patchset implements the KHR_debug extension.
The series looks good to me. Just a few minor comments to follow.
Reviewed-by: Brian Paul bri...@vmware.com
___
mesa-dev mailing list
On 08/26/2013 04:43 AM, Timothy Arceri wrote:
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/hash.c | 26 ++
src/mesa/main/hash.h |3 +++
2 files changed, 29 insertions(+)
diff --git a/src/mesa/main/hash.c b/src/mesa/main/hash.c
index
On 08/26/2013 04:43 AM, Timothy Arceri wrote:
[...]
@@ -722,13 +769,112 @@ void GLAPIENTRY
_mesa_PushDebugGroup(GLenum source, GLuint id, GLsizei length,
const GLchar *message)
{
+ const char *callerstr = glPushDebugGroup;
+ int s, t, sev;
+ GLint prevStackDepth;
On 08/26/2013 04:43 AM, Timothy Arceri wrote:
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/objectlabel.c | 277 +++
src/mesa/main/objectlabel.h | 61 ++
2 files changed, 338 insertions(+)
create mode 100644
On 22 August 2013 13:58, Kenneth Graunke kenn...@whitecape.org wrote:
On 08/20/2013 11:30 AM, Paul Berry wrote:
From: Eric Anholt e...@anholt.net
All but two of the piglit GLSL 1.50/uniform_buffer tests work, and
maxuniformblocksize and referenced-by-shader work.
v2 (Paul Berry
On 23 August 2013 14:55, Kenneth Graunke kenn...@whitecape.org wrote:
On 08/23/2013 11:50 AM, Paul Berry wrote:
If brwNewProgram is asked to create a program for an unrecognized
target, don't bother falling back on _mesa_new_program(). That just
hides bugs.
---
On 08/23/2013 03:10 PM, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
There's just no way resetting the counters is working with nested/overlapping
queries.
---
src/gallium/drivers/softpipe/sp_prim_vbuf.c |2 +-
src/gallium/drivers/softpipe/sp_query.c | 33
to silence gcc 4.8.1 warnings. And improve the ASSERT(0) call.
---
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 ddfeee2..1052f75 100644
--- a/src/mesa/main/macros.h
+++
---
src/mesa/swrast/s_texfilter.c |1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/swrast/s_texfilter.c b/src/mesa/swrast/s_texfilter.c
index 0920a64..312b9ce 100644
--- a/src/mesa/swrast/s_texfilter.c
+++ b/src/mesa/swrast/s_texfilter.c
@@ -274,6 +274,7 @@
---
src/egl/eglut/eglut_wayland.c | 129 +++---
1 file changed, 97 insertions(+), 32 deletions(-)
diff --git a/src/egl/eglut/eglut_wayland.c b/src/egl/eglut/eglut_wayland.c
index 61207d2..809d8b4 100644
--- a/src/egl/eglut/eglut_wayland.c
+++
From: Julien Cristau jcris...@debian.org
This isn't used anywhere.
v2: Rebase to the top of the patch series (anholt)
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1)
Signed-off-by: Julien Cristau jcris...@debian.org
Signed-off-by: Eric Anholt e...@anholt.net
---
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/hostx.c | 40 +++-
1 file changed, 27 insertions(+), 13
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/hostx.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/hostx.c | 45 -
1 file changed, 32
From: Julien Cristau jcris...@debian.org
v2: Dropped the hostx_load_keymap changes, now that that function is
gutted (anholt).
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1)
Signed-off-by: Julien Cristau jcris...@debian.org
Signed-off-by: Eric Anholt e...@anholt.net
---
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/hostx.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/hostx.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/hostx.c | 59 +++--
1 file changed, 33
From: Julien Cristau jcris...@debian.org
v2: Also remove the prototype (anholt)
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1)
Signed-off-by: Julien Cristau jcris...@debian.org
Signed-off-by: Eric Anholt e...@anholt.net
---
hw/kdrive/ephyr/hostx.c | 15 ---
From: Julien Cristau jcris...@debian.org
v2: Rebase on indentation changes, squash in a simpler variant of the
later event compression patch, fix server hang or segfault on
window close by reimplementing the x_io_error_handler in the new
XCB event loop (anholt).
Reviewed-by: Mikhail
From: Julien Cristau jcris...@debian.org
Require new xcb-xf86dri to get fixed GetDrawableInfo.
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
configure.ac | 2 +-
hw/kdrive/ephyr/ephyrdri.c | 201
From: Julien Cristau jcris...@debian.org
v2: Massive, conflict-filled rebase. I think I resolved everything
right, but this host xv code makes my eyes bleed. Touch-tested XV
after the rebase (anholt).
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1)
Signed-off-by: Julien
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/ephyrhostproxy.c | 91 ---
hw/kdrive/ephyr/ephyrhostproxy.h |
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
hw/kdrive/ephyr/Makefile.am | 2 -
hw/kdrive/ephyr/XF86dri.c | 654
hw/kdrive/ephyr/xf86dri.h |
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
configure.ac| 2 +-
hw/kdrive/ephyr/hostx.c | 13 -
2 files changed, 9
From: Julien Cristau jcris...@debian.org
Signed-off-by: Julien Cristau jcris...@debian.org
Reviewed-by: Eric Anholt e...@anholt.net
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
---
hw/kdrive/ephyr/hostx.c | 15 +++
1 file changed, 15 insertions(+)
diff --git
From: Julien Cristau jcris...@debian.org
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net
Reviewed-by: Eric Anholt e...@anholt.net
Signed-off-by: Julien Cristau jcris...@debian.org
---
configure.ac| 4 ++--
hw/kdrive/ephyr/hostx.c | 64
From: Julien Cristau jcris...@debian.org
v2: Rebase on master, notably adding XCB for
X_GLXvop_MakeCurrentReadSGI (anholt).
Reviewed-by: Mikhail Gusarov dotted...@dottedmag.net (v1)
Signed-off-by: Julien Cristau jcris...@debian.org
Signed-off-by: Eric Anholt e...@anholt.net
---
From: Julien Cristau jcris...@debian.org
Drop obsolete comments about interaction between Xlib code and that
macro, and stop undefining it.
Signed-off-by: Julien Cristau jcris...@debian.org
Reviewed-by: Eric Anholt e...@anholt.net
---
hw/kdrive/ephyr/ephyrhostglx.c | 11 ---
On 08/24/2013 10:41 AM, Francisco Jerez wrote:
Chad Versace chad.vers...@linux.intel.com writes:
On 08/23/2013 02:18 PM, Paul Berry wrote:
The disadvantages are that (a) we need an explicit enum value for 0,
and (b) we can't use related operators like |= unless we define
additional
Did this patch series go to the wrong list? Did you intend xorg-devel
instead?
On 08/26/2013 11:16 AM, Eric Anholt wrote:
From: Julien Cristau jcris...@debian.org
This isn't used anywhere.
v2: Rebase to the top of the patch series (anholt)
Reviewed-by: Mikhail Gusarov
Series is
Reviewed-by: Chad Versace chad.vers...@linux.intel.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Hi
15.08.2013 13:54, Marek Olšák wrote:
On Thu, Aug 15, 2013 at 10:27 AM, Christian König
deathsim...@vodafone.de wrote:
Am 15.08.2013 05:25, schrieb Marek Olšák:
(This should be applied before MSAA, which will need to be rebased.)
It moves all sampler view descriptors to a buffer.
It
---
configure.ac | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index 40e052c..e20a228 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1925,8 +1925,8 @@ AM_CONDITIONAL(NEED_GALLIUM_SOFTPIPE_DRIVER, test
x$HAVE_GALLIUM_SVGA = xyes -
---
configure.ac | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index 40e052c..e20a228 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1925,8 +1925,8 @@ AM_CONDITIONAL(NEED_GALLIUM_SOFTPIPE_DRIVER, test
x$HAVE_GALLIUM_SVGA = xyes -
Ian Romanick i...@freedesktop.org writes:
[...]
Disadvantage (b) can be made painless with the macro I discuss below.
IMHO it would be nicer to define generic templates implementing
overloads for all bitwise operators. They would have to reference the
bitmask_enumeration_traits structure
Ian Romanick i...@freedesktop.org writes:
Did this patch series go to the wrong list? Did you intend xorg-devel
instead?
Yes, I intended the other list, just mis-typed.
pgpiOoE6RwZ_a.pgp
Description: PGP signature
___
mesa-dev mailing list
Kenneth Graunke kenn...@whitecape.org writes:
Previously, we allocated space in brw_vs_prog_data's params and
pull_params arrays for MAX_CLIP_PLANES vec4s---even when it wasn't
necessary.
On a 64-bit architecture, this used 0.5 kB of space (8 clip planes *
4 floats per plane * 8 bytes per
On 23 August 2013 17:45, Matt Turner matts...@gmail.com wrote:
On Fri, Aug 23, 2013 at 8:27 AM, Paul Berry stereotype...@gmail.com
wrote:
On 22 August 2013 16:08, Matt Turner matts...@gmail.com wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
On 25 August 2013 23:46, Kenneth Graunke kenn...@whitecape.org wrote:
Previously, we allocated space in brw_vs_prog_data's params and
pull_params arrays for MAX_CLIP_PLANES vec4s---even when it wasn't
necessary.
On a 64-bit architecture, this used 0.5 kB of space (8 clip planes *
4 floats
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68460
Cc: 9.2 mesa-sta...@lists.freedesktop.org
---
src/glsl/ast_to_hir.cpp | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index feff586..bb25d81 100644
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68504
Bugzilla: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58241
CC: mesa-sta...@lists.freedesktop.org
---
configure.ac | 11 +++
src/gallium/state_trackers/clover/Makefile.am | 2 +-
2 files
From: Kris em...@zero-io.net
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68544
CC: mesa-sta...@lists.freedesktop.org
---
configure.ac | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index 40e052c..f135f81 100644
---
https://bugs.freedesktop.org/show_bug.cgi?id=68504
Chí-Thanh Christopher Nguyễn chith...@gentoo.org changed:
What|Removed |Added
Status|RESOLVED
This is a re-spin of patches 23-30 of my series i965/gen7: Initial
geometry shader support. from 8/20. Based on code review, I've made
the following changes:
- Rather than using vec4_gs to refer to the new user-programmable
geometry shader functionality, and gs to refer to the old
Now that the name gs is no longer used to refer to the legacy fixed
function geometry shaders, we can use it to refer to user-defined
geometry shaders.
---
src/mesa/drivers/dri/i965/brw_context.h | 4 ++--
src/mesa/drivers/dri/i965/brw_vec4_gs_visitor.cpp | 2 +-
ff is for fixed function. This frees up the name gs to refer to
user-defined geometry shaders.
---
src/mesa/drivers/dri/i965/brw_context.h | 14 +--
src/mesa/drivers/dri/i965/brw_gs.c | 42
src/mesa/drivers/dri/i965/brw_gs.h | 19 ++--
This will avoid confusion when we add geometry shaders, since these
data structures will be shared by vertex and geometry shaders.
---
src/mesa/drivers/dri/i965/brw_context.h| 31 --
.../drivers/dri/i965/brw_vec4_reg_allocate.cpp | 31 +++---
2
Defines that previously referred to VS now refer to VEC4, since they
will be shared by the user-programmable vertex shader and geometry
shader stages.
Defines that previously referred to the Gen6 geometry shader stage
(which is only used for transform feedback) are now renamed to
explicitly refer
This paves the way for sharing the code that will set up the vertex
and geometry shader pipeline state.
---
src/mesa/drivers/dri/i965/brw_context.h | 47 ++--
src/mesa/drivers/dri/i965/brw_draw.c | 3 +-
src/mesa/drivers/dri/i965/brw_misc_state.c |
v2: Change name from vec4_gs to simply gs.
---
src/mesa/drivers/dri/i965/brw_context.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h
b/src/mesa/drivers/dri/i965/brw_context.h
index 9784956..d5ca4dd 100644
---
---
src/mesa/drivers/dri/i965/brw_context.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h
b/src/mesa/drivers/dri/i965/brw_context.h
index d5ca4dd..be5175f 100644
--- a/src/mesa/drivers/dri/i965/brw_context.h
+++
Previously, we gave all of the URB space (other than the small amount
that is used for push constants) to the vertex shader. However, when
a geometry shader is active, we need to divide it up between the
vertex and geometry shaders.
The size of the URB entries for the vertex and geometry shaders
Previously, we would always use the same push constant allocation
regardless of what shader programs were being run: the available push
constant space was split into 2 equal size partitions, one for the
vertex shader, and one for the fragment shader.
Now that we are adding geometry shader
The hardware requires that after constant buffers for a stage are
allocated using a 3DSTATE_PUSH_CONSTANT_ALLOC_{VS,HS,DS,GS,PS}
command, and prior to execution of a 3DPRIMITIVE, the corresponding
stage's constant buffers must be reprogrammed using a
3DSTATE_CONSTANT_{VS,HS,DS,GS,PS} command.
---
src/mesa/drivers/dri/i965/brw_state.h| 8 +++
src/mesa/drivers/dri/i965/brw_vs_surface_state.c | 66 +++-
2 files changed, 50 insertions(+), 24 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
b/src/mesa/drivers/dri/i965/brw_state.h
index
---
src/mesa/drivers/dri/i965/brw_state.h| 6
src/mesa/drivers/dri/i965/brw_vs_surface_state.c | 42
2 files changed, 35 insertions(+), 13 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
b/src/mesa/drivers/dri/i965/brw_state.h
index
This patch implements pull constant upload, binding table upload, and
surface setup for geometry shaders, by re-using vertex shader code
that was generalized in previous patches.
Based on work by Eric Anholt e...@anholt.net.
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
v2: Use unsigned rather than GLuint.
---
src/mesa/drivers/dri/i965/Makefile.sources| 1 +
src/mesa/drivers/dri/i965/brw_defines.h | 10 +
src/mesa/drivers/dri/i965/brw_draw.c | 2 +
---
src/mesa/drivers/dri/i965/brw_context.h | 7 +
src/mesa/drivers/dri/i965/gen6_vs_state.c | 45 ---
2 files changed, 36 insertions(+), 16 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h
b/src/mesa/drivers/dri/i965/brw_context.h
index
Previously these functions would accept a pointer to the binding table
and an index indicating which entry in the binding table should be
updated. Now they merely take a pointer to the binding table entry to
be updated.
This will make it easier to generalize brw_texture_surfaces to support
There is a slight functionality change. Previously we would compute a
common value for num_samplers for all stages, and populate that many
entries in each stage's surf_offset table regardless of how many
samplers each stage used. Now we only populate the number of entries
in the surf_offset
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
index 1b4388c..600b564 100644
--- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
---
src/mesa/drivers/dri/i965/brw_state.h| 1 +
src/mesa/drivers/dri/i965/brw_state_upload.c | 1 +
src/mesa/drivers/dri/i965/brw_wm_sampler_state.c | 28
3 files changed, 30 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
Dword 2 of all 3DSTATE commands is the same, so there's no need tohave
separate defines for it. This will allow us to unify some of the
state setup code between VS and GS.
---
src/mesa/drivers/dri/i965/brw_defines.h | 30 +-
src/mesa/drivers/dri/i965/gen6_blorp.cpp
---
src/mesa/drivers/dri/i965/brw_state.h | 41 ++
src/mesa/drivers/dri/i965/gen7_vs_state.c | 123 --
2 files changed, 122 insertions(+), 42 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
b/src/mesa/drivers/dri/i965/brw_state.h
index
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
src/mesa/drivers/dri/i965/brw_defines.h | 7 ++
src/mesa/drivers/dri/i965/brw_state.h| 2 +
src/mesa/drivers/dri/i965/brw_state_upload.c | 2 +
src/mesa/drivers/dri/i965/gen7_disable.c | 33 ---
Chí-Thanh Christopher Nguyễn chith...@gentoo.org writes:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68504
Bugzilla: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58241
CC: mesa-sta...@lists.freedesktop.org
---
configure.ac | 11 +++
Sorry, I missed that the author already sent his patch to the list. Please
disregard.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Chí-Thanh Christopher Nguyễn schrieb:
How about disabling AltiVec code generation by adding -mno-altivec to
CXXFLAGS? AFAICT what's going on is that the -maltivec option (which
is implied by your -mcpu=7450) enables a number of language
extensions as a side effect that collide with the C++
Chí-Thanh Christopher Nguyễn chith...@gentoo.org writes:
Francisco Jerez schrieb:
[...]
How about disabling AltiVec code generation by adding -mno-altivec to
CXXFLAGS? AFAICT what's going on is that the -maltivec option (which
is implied by your -mcpu=7450) enables a number of language
Chí-Thanh Christopher Nguyễn chith...@gentoo.org writes:
Chí-Thanh Christopher Nguyễn schrieb:
How about disabling AltiVec code generation by adding -mno-altivec to
CXXFLAGS? AFAICT what's going on is that the -maltivec option (which
is implied by your -mcpu=7450) enables a number of
Francisco Jerez schrieb:
Chí-Thanh Christopher Nguyễn chith...@gentoo.org writes:
Chí-Thanh Christopher Nguyễn schrieb:
How about disabling AltiVec code generation by adding
-mno-altivec to CXXFLAGS? AFAICT what's going on is that the
-maltivec option (which is implied by your -mcpu=7450)
On 08/26/2013 02:16 PM, Matt Turner wrote:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68460
Cc: 9.2 mesa-sta...@lists.freedesktop.org
---
src/glsl/ast_to_hir.cpp | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp
On Mon, Aug 26, 2013 at 4:35 PM, Ian Romanick i...@freedesktop.org wrote:
I think this will allow things like:
struct foo {
layout(row_major) mat4 bar;
};
Yeah?
Don't think so:
error: syntax error, unexpected LAYOUT_TOK
___
mesa-dev mailing
On 08/26/2013 01:10 PM, Francisco Jerez wrote:
Ian Romanick i...@freedesktop.org writes:
[...]
Disadvantage (b) can be made painless with the macro I discuss below.
IMHO it would be nicer to define generic templates implementing
overloads for all bitwise operators. They would have to
On 08/26/2013 04:49 PM, Matt Turner wrote:
On Mon, Aug 26, 2013 at 4:35 PM, Ian Romanick i...@freedesktop.org wrote:
I think this will allow things like:
struct foo {
layout(row_major) mat4 bar;
};
Yeah?
Don't think so:
error: syntax error, unexpected LAYOUT_TOK
Right... because
1 - 100 of 117 matches
Mail list logo