[Mesa-dev] [Bug 49717] SCons build error with clang

2012-05-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49717 --- Comment #4 from Vinson Lee 2012-05-12 22:38:20 PDT --- (In reply to comment #3) > I suspect so. Is $CXX set to clang++? Yes, $CXX is set to clang++. SCons build with clang also fails at the same place on Mac OS X. -- Configure bugmail: h

Re: [Mesa-dev] [PATCH 10/11] clover: Add function for building a clover::module for non-TGSI targets

2012-05-12 Thread Tom Stellard
On Sun, May 13, 2012 at 12:40:43AM +0200, Francisco Jerez wrote: > Tom Stellard writes: > > > --- > > .../state_trackers/clover/llvm/invocation.cpp | 174 > > +++- > > 1 files changed, 165 insertions(+), 9 deletions(-) > > > > diff --git a/src/gallium/state_trackers/clover

Re: [Mesa-dev] [PATCH 10/11] clover: Add function for building a clover::module for non-TGSI targets

2012-05-12 Thread Francisco Jerez
Tom Stellard writes: > --- > .../state_trackers/clover/llvm/invocation.cpp | 174 > +++- > 1 files changed, 165 insertions(+), 9 deletions(-) > > diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp > b/src/gallium/state_trackers/clover/llvm/invocation.cpp >

Re: [Mesa-dev] [PATCH] clover, gallium: add PIPE_COMPUTE_CAP_MAX_THREADS_PER_BLOCK

2012-05-12 Thread Francisco Jerez
Christoph Bumiller writes: > This is not necessarily the product of MAX_BLOCK_SIZE[i]. Thanks, looks good. I've pushed it along with a fix for clEnqueueNDRangeKernel() to check that the work-group dimensions specified by the user are within the limit set by this cap. > --- > src/gallium/docs/

Re: [Mesa-dev] [PATCH 1/3] gallium/util: remove unused parameter nr_vertex_buffers in util_draw_max_index

2012-05-12 Thread Jose Fonseca
- Original Message - > On Sat, May 12, 2012 at 6:28 PM, Jose Fonseca > wrote: > > This may come handy in the future, so instead of changing the > > prototype, please just add a (void)nr_vertex_buffers in the body. > > It doesn't produce a warning, so I'll just leave it the way it is. >

Re: [Mesa-dev] [PATCH 1/3] gallium/util: remove unused parameter nr_vertex_buffers in util_draw_max_index

2012-05-12 Thread Marek Olšák
On Sat, May 12, 2012 at 6:28 PM, Jose Fonseca wrote: > This may come handy in the future, so instead of changing the prototype, > please just add a (void)nr_vertex_buffers in the body. It doesn't produce a warning, so I'll just leave it the way it is. FWIW, once we add the start_slot parameter

Re: [Mesa-dev] [PATCH 3/3] gallium: remove user_buffer_create from the interface

2012-05-12 Thread Jose Fonseca
Patch 2 and 3 look good to me. I think this is a nice cleanup. Jose - Original Message - > Nothing uses it now. > --- > src/gallium/auxiliary/util/u_inlines.h |8 - > src/gallium/auxiliary/util/u_slab.h |2 +- > src/gallium/drivers/galahad/glhd_screen.c|

Re: [Mesa-dev] [PATCH 1/3] gallium/util: remove unused parameter nr_vertex_buffers in util_draw_max_index

2012-05-12 Thread Jose Fonseca
This may come handy in the future, so instead of changing the prototype, please just add a (void)nr_vertex_buffers in the body. Joe - Original Message - > --- > src/gallium/auxiliary/draw/draw_pt.c |1 - > src/gallium/auxiliary/util/u_draw.c |1 - > src/gallium/auxiliary/util/u

[Mesa-dev] [PATCH] mesa: initialize renderbuffer fields even if AllocStorage fails

2012-05-12 Thread Marek Olšák
It may fail with an unsupported format, but that's not an allowed case where RenderbufferStorage may fail. And on success, we should return valid values from GetRenderbufferParameteriv. This fixes piglit: get-renderbuffer-internalformat --- src/mesa/main/fbobject.c | 18 +- 1 fi

[Mesa-dev] [PATCH 3/3] gallium: remove user_buffer_create from the interface

2012-05-12 Thread Marek Olšák
Nothing uses it now. --- src/gallium/auxiliary/util/u_inlines.h |8 - src/gallium/auxiliary/util/u_slab.h |2 +- src/gallium/drivers/galahad/glhd_screen.c| 23 -- src/gallium/drivers/i915/i915_resource.c |1 - src/gallium/drivers/identity/id_scr

[Mesa-dev] [PATCH 2/3] gallium/graw: stop using user_buffer_create

2012-05-12 Thread Marek Olšák
This is compile-tested. --- src/gallium/auxiliary/util/u_inlines.h | 13 + src/gallium/tests/graw/fs-fragcoord.c|9 + src/gallium/tests/graw/fs-frontface.c|9 + src/gallium/tests/graw/fs-test.c |9 + src/gallium/tests/graw/fs-writ

[Mesa-dev] [PATCH 1/3] gallium/util: remove unused parameter nr_vertex_buffers in util_draw_max_index

2012-05-12 Thread Marek Olšák
--- src/gallium/auxiliary/draw/draw_pt.c |1 - src/gallium/auxiliary/util/u_draw.c |1 - src/gallium/auxiliary/util/u_draw.h |1 - 3 files changed, 3 deletions(-) diff --git a/src/gallium/auxiliary/draw/draw_pt.c b/src/gallium/auxiliary/draw/draw_pt.c index 025d539..dbe5e94 100644

[Mesa-dev] [Bug 49830] New: [compile error] Makefile:29: *** target file `default' has both : and :: entries. Stop.

2012-05-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49830 Bug #: 49830 Summary: [compile error] Makefile:29: *** target file `default' has both : and :: entries. Stop. Classification: Unclassified Product: Mesa Version: git

Re: [Mesa-dev] [PATCH] mesa: Use the proper feature test macros for strtod_l and strtof[_l].

2012-05-12 Thread Bryan Henderson
>Tell that to the config files. It was added to darwin's config file, >presumably because we supported the functionality it enabled. I guess there are a number of things this sentence could mean. Based on the juxtaposition of -D_GNU_SOURCE with -D_BSD_SOURCE and -D_DARWIN_C_SOURCE, this suggests