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

2012-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49717 Bug #: 49717 Summary: SCons build error with clang Classification: Unclassified Product: Mesa Version: git Platform: x86-64 (AMD64) OS/Version: All Status: NEW

Re: [Mesa-dev] [PATCH 2/3] lp_bld_arit: Fix lp_build_sgn.

2012-05-09 Thread Olivier Galibert
On Wed, May 09, 2012 at 10:01:37AM -0700, Jose Fonseca wrote: > Something else is not right here, as > > lp_build_const_vec(bld->gallivm, type, -1.0) > > and > > lp_build_const_int_vec(bld->gallivm, type, -1) > > should produce the same output for unnormalized signed integer types. > > Wh

[Mesa-dev] [Bug 49703] incorrect macros #define M(row, col) m[col*4+row]

2012-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49703 --- Comment #2 from Chuprin Vladislav 2012-05-09 21:14:08 PDT --- Sorry I did not know it. Thank you for your answer. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail because: -

Re: [Mesa-dev] [PATCH 5/5] mesa: add DEBUG_INCOMPLETE_TEXTURE, DEBUG_INCOMPLETE_FBO flags

2012-05-09 Thread Eric Anholt
On Wed, 9 May 2012 13:53:52 -0600, Brian Paul wrote: > Instead of having to hack the code to enable these debugging options, > set them through the MESA_DEBUG env var. I was thinking "why don't we just always print this debug when someone tries to use an incomplete texture?", but then these inco

Re: [Mesa-dev] [PATCH 3/3] i965: Completely annotate the batch bo when aub dumping.

2012-05-09 Thread Eric Anholt
On Mon, 7 May 2012 14:32:18 -0700, Paul Berry wrote: > +/** > + * Generate a set of aub file annotations for the current batch buffer, and > + * deliver them to DRM. > + * > + * The "used" section of the batch buffer (the portion containing batch > + * commands) is annotated with AUB_TRACE_TYPE_B

Re: [Mesa-dev] [PATCH 3/5] i965/fs: Remove the requirement of no dead code for interference checks.

2012-05-09 Thread Eric Anholt
On Wed, 09 May 2012 00:43:47 -0700, Kenneth Graunke wrote: > On 05/08/2012 05:59 PM, Eric Anholt wrote: > > This will be convenient when I want to comment out optimization code > > to see the raw program being optimized, but more importantly will let > > the interference check be used during opti

Re: [Mesa-dev] [PATCH 5/5] glsl: Improve the local dead code optimization to eliminate unused channels.

2012-05-09 Thread Eric Anholt
On Wed, 09 May 2012 08:46:30 -0700, Ian Romanick wrote: > On 05/08/2012 05:59 PM, Eric Anholt wrote: > > Total instructions: 261582 -> 261316 > > 135/2147 programs affected (6.3%) > > 36752 -> 36486 instructions in affected programs (0.7% reduction) > > > > This excludes a tropics shader that no

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

2012-05-09 Thread Chad Versace
On 05/09/2012 12:52 PM, Kenneth Graunke wrote: > From: Bryan Henderson > > [v2/Kayden: rebased version of Bryan's original patch from: > https://bugs.freedesktop.org/show_bug.cgi?id=33447] > > Cc: Jeremy Huddleston > Cc: Chad Versace > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=33

Re: [Mesa-dev] libGL.so.* are not installed when configured with "--enable-xlib-glx"

2012-05-09 Thread Sven Joachim
On 2012-05-05 19:17 +0200, Sven Joachim wrote: > On 2012-05-03 11:34 +0200, Sven Joachim wrote: > >> When configuring mesa git master with "--enable-xlib-glx", the GL >> library does not get installed. > > Bisection showed that this has been broken by commit 8ffb098234: > > commit 8ffb098234fbc3d2

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

2012-05-09 Thread Jeremy Huddleston
No, that's not right. We don't use glibc, and we support strtod_l, strtof_l, and strtof. This change would do the wrong thing for __APPLE__. Perhaps something more like this: #if defined(__APPLE__) || (defined(__GLIBC__) && __GLIBC__ >= 2 && __GLIBC_MINOR__ > 2 && !defined(__CYGWIN__) && !d

[Mesa-dev] [PATCH 5/5] mesa: add DEBUG_INCOMPLETE_TEXTURE, DEBUG_INCOMPLETE_FBO flags

2012-05-09 Thread Brian Paul
Instead of having to hack the code to enable these debugging options, set them through the MESA_DEBUG env var. --- src/mesa/main/debug.c|2 ++ src/mesa/main/fbobject.c | 20 ++-- src/mesa/main/mtypes.h |2 ++ src/mesa/main/texobj.c | 17 + 4 file

[Mesa-dev] [PATCH 4/5] mesa: implement DEBUG_ALWAYS_FLUSH debug option

2012-05-09 Thread Brian Paul
This flag has been around for a while but it wasn't actually used anywhere. Now, setting this flag causes a glFlush() to be issued after each drawing call (including glBegin/End, glDrawElements, glDrawArrays, glDrawPixels, glCopyPixels and glBitmap). --- src/mesa/main/drawpix.c | 12 +

[Mesa-dev] [PATCH 3/5] mesa: define DEBUG_SILENT flag, use in output_if_debug()

2012-05-09 Thread Brian Paul
--- src/mesa/main/debug.c |1 + src/mesa/main/errors.c | 21 ++--- src/mesa/main/mtypes.h |3 ++- 3 files changed, 13 insertions(+), 12 deletions(-) diff --git a/src/mesa/main/debug.c b/src/mesa/main/debug.c index 5f37f37..0e73fcb 100644 --- a/src/mesa/main/debug.c +++

[Mesa-dev] [PATCH 2/5] mesa: clean-up the debug/verbose flag setup code

2012-05-09 Thread Brian Paul
Split the verbose and debug flag setup code into separate functions. --- src/mesa/main/debug.c | 72 +++- 1 files changed, 46 insertions(+), 26 deletions(-) diff --git a/src/mesa/main/debug.c b/src/mesa/main/debug.c index f7b1f71..5f37f37 100644 --- a

[Mesa-dev] [PATCH 1/5] mesa: do FLUSH_VERTICES() in _mesa_flush/finish()

2012-05-09 Thread Brian Paul
This was being done in the _mesa_Flush/Finish() calls but if there was an internal call to _mesa_flush/finish() the FLUSH_VERTICES() wouldn't happen. Looks like only the intel and radeon drivers made such calls in MakeCurrent(). --- src/mesa/main/context.c |6 -- 1 files changed, 4 insert

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

2012-05-09 Thread Kenneth Graunke
From: Bryan Henderson [v2/Kayden: rebased version of Bryan's original patch from: https://bugs.freedesktop.org/show_bug.cgi?id=33447] Cc: Jeremy Huddleston Cc: Chad Versace Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=33447 Signed-off-by: Kenneth Graunke --- src/glsl/strtod.c

[Mesa-dev] [Bug 49703] incorrect macros #define M(row, col) m[col*4+row]

2012-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49703 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[Mesa-dev] [Bug 49703] New: incorrect macros #define M(row, col) m[col*4+row]

2012-05-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=49703 Bug #: 49703 Summary: incorrect macros #define M(row,col) m[col*4+row] Classification: Unclassified Product: Mesa Version: 8.0 Platform: All OS/Version: All Status: N

Re: [Mesa-dev] [PATCH 2/3] lp_bld_arit: Fix lp_build_sgn.

2012-05-09 Thread Jose Fonseca
Something else is not right here, as lp_build_const_vec(bld->gallivm, type, -1.0) and lp_build_const_int_vec(bld->gallivm, type, -1) should produce the same output for unnormalized signed integer types. Which means the error is likely somewhere else: - lp_build_const_vec is buggy - the ty

Re: [Mesa-dev] [PATCH] svga: implement CEIL opcode translation

2012-05-09 Thread Jose Fonseca
Looks good to me. Jose - Original Message - > --- > src/gallium/drivers/svga/svga_tgsi_insn.c | 28 > > 1 files changed, 28 insertions(+), 0 deletions(-) > > diff --git a/src/gallium/drivers/svga/svga_tgsi_insn.c > b/src/gallium/drivers/svga/svga_tgsi_ins

[Mesa-dev] [PATCH] svga: implement CEIL opcode translation

2012-05-09 Thread Brian Paul
--- src/gallium/drivers/svga/svga_tgsi_insn.c | 28 1 files changed, 28 insertions(+), 0 deletions(-) diff --git a/src/gallium/drivers/svga/svga_tgsi_insn.c b/src/gallium/drivers/svga/svga_tgsi_insn.c index 5e6d1fb..a689126 100644 --- a/src/gallium/drivers/svga/svg

Re: [Mesa-dev] [PATCH] r600g: Added CEIL to r600_bytecode_get_num_operands

2012-05-09 Thread Alex Deucher
On Wed, May 9, 2012 at 11:31 AM, Kai Wasserbäch wrote: > > Signed-off-by: Kai Wasserbäch added as part of 6ae12bac596ce3a6aa8e09f638ad2cb4a7c18e5c Alex > --- >  Wasn't able to build-test this one, my connection is a little slow and I > would >  need to update my build chroot on this machine.

Re: [Mesa-dev] [PATCH 5/5] glsl: Improve the local dead code optimization to eliminate unused channels.

2012-05-09 Thread Ian Romanick
On 05/08/2012 05:59 PM, Eric Anholt wrote: Total instructions: 261582 -> 261316 135/2147 programs affected (6.3%) 36752 -> 36486 instructions in affected programs (0.7% reduction) This excludes a tropics shader that now gets 16-wide mode and throws off the numbers. 5 shaders are hurt: two ext

[Mesa-dev] [PATCH] r600g: Added CEIL to r600_bytecode_get_num_operands

2012-05-09 Thread Kai Wasserbäch
Signed-off-by: Kai Wasserbäch --- Wasn't able to build-test this one, my connection is a little slow and I would need to update my build chroot on this machine. src/gallium/drivers/r600/r600_asm.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/r600/r600_asm.c b/

Re: [Mesa-dev] [PATCH] r600g: Handle TGSI_OPCODE_CEIL (v2)

2012-05-09 Thread Tom Stellard
On Mon, May 07, 2012 at 05:59:35PM +0200, Kai Wasserbäch wrote: > v2: Enabled CEIL on Cayman too. > > Signed-off-by: Kai Wasserbäch > Reviewed-by: Tom Stellard > --- > Thanks for the review, Tom! > > If this patch is accepted, please apply it to master for me, I don't have > commit access on

Re: [Mesa-dev] [PATCH 1/5] i965/fs: When doing no work for live interval calculation, do no allocation.

2012-05-09 Thread Kenneth Graunke
On 05/08/2012 05:59 PM, Eric Anholt wrote: When I had a bug causing the backend to never finish optimizing, it also sent me deep into swap. This avoids extra memory allocation per trip through optimization, and thus may reduce the peak memory allocation of the driver even in the success case. --

Re: [Mesa-dev] [PATCH 3/5] i965/fs: Remove the requirement of no dead code for interference checks.

2012-05-09 Thread Kenneth Graunke
On 05/08/2012 05:59 PM, Eric Anholt wrote: This will be convenient when I want to comment out optimization code to see the raw program being optimized, but more importantly will let the interference check be used during optimization. --- .../drivers/dri/i965/brw_fs_live_variables.cpp | 24

Re: [Mesa-dev] [PATCH 2/5] i965/fs: Add support for copy propagation.

2012-05-09 Thread Kenneth Graunke
On 05/08/2012 05:59 PM, Eric Anholt wrote: We could do more by handling abs/negate and non-GRF sources, but this is a good start. Total instructions: 208032 -> 207184 60/1246 programs affected (4.8%) 23286 -> 22438 instructions in affected programs (3.6% reduction) --- src/mesa/drivers/dri/i

Re: [Mesa-dev] [PATCH 0.3] llvmpipe iabs/isgn support

2012-05-09 Thread Dave Airlie
On Wed, May 9, 2012 at 7:26 AM, Olivier Galibert wrote: >  Hi, > > These three patches make integer abs and sign work for llvmpipe. snap :-) http://cgit.freedesktop.org/~airlied/mesa/commit/?h=llvmpipe-int-workv3&id=860a65200f3ec7d35dc5ccd59b632ead2a27da50 so I should just pick over that one :0