Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Christian König
Am 24.08.2013 03:30, schrieb Vadim Girlin: Currently llvm backend always exports at least one color in pixel shader even if no color buffers are enabled. With depth/stencil exports this can result in the following code: EXPORT PIXEL 0 R0.xyzw VPM EXPORT PIXEL 61

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Christoph Bumiller
On 24.08.2013 11:44, Christian König wrote: Am 24.08.2013 03:30, schrieb Vadim Girlin: Currently llvm backend always exports at least one color in pixel shader even if no color buffers are enabled. With depth/stencil exports this can result in the following code: EXPORT PIXEL 0

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Marek Olšák
Like Christoph said, COLOR0 (if available) must always be exported for alpha test. Marek On Sat, Aug 24, 2013 at 3:30 AM, Vadim Girlin vadimgir...@gmail.com wrote: Currently llvm backend always exports at least one color in pixel shader even if no color buffers are enabled. With depth/stencil

[Mesa-dev] [Bug 68504] New: 9.2-rc1 clover fails to build on ppc: cannot convert 'bool' to '__vector(4) __bool int' in return

2013-08-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68504 Priority: medium Bug ID: 68504 Assignee: mesa-dev@lists.freedesktop.org Summary: 9.2-rc1 clover fails to build on ppc: cannot convert 'bool' to '__vector(4) __bool int' in return

[Mesa-dev] [Bug 68504] 9.2-rc1 clover fails to build on ppc: cannot convert 'bool' to '__vector(4) __bool int' in return

2013-08-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68504 Chí-Thanh Christopher Nguyễn chith...@gentoo.org changed: What|Removed |Added Attachment #84560|text/plain

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Vadim Girlin
On 08/24/2013 02:31 PM, Marek Olšák wrote: Like Christoph said, COLOR0 (if available) must always be exported for alpha test. Are there any piglit tests for that? I didn't see any regressions with this patch (at least on evergreen), possibly I messed up the testing somehow. Also I think old

Re: [Mesa-dev] [PATCH] nouveau/video: avoid overwriting base codec init with template

2013-08-24 Thread Ilia Mirkin
On Sat, Aug 24, 2013 at 8:36 AM, Martin Peres martin.pe...@labri.fr wrote: On 24/08/2013 11:38, Christian König wrote: Am 24.08.2013 06:50, schrieb Ilia Mirkin: Commit 53e20b8b introduced the use of a template to initialize some common fields. Move this copying of fields to before the common

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Alex Deucher
On Sat, Aug 24, 2013 at 5:44 AM, Christian König deathsim...@vodafone.de wrote: Am 24.08.2013 03:30, schrieb Vadim Girlin: Currently llvm backend always exports at least one color in pixel shader even if no color buffers are enabled. With depth/stencil exports this can result in the following

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Marek Olšák
See piglit/fbo-alphatest-nocolor. Marek On Sat, Aug 24, 2013 at 3:12 PM, Vadim Girlin vadimgir...@gmail.com wrote: On 08/24/2013 02:31 PM, Marek Olšák wrote: Like Christoph said, COLOR0 (if available) must always be exported for alpha test. Are there any piglit tests for that? I didn't

Re: [Mesa-dev] [Mesa-announce] Mesa 9.2 release candidate 2

2013-08-24 Thread Sven Joachim
On 2013-08-23 09:48 +0200, Andreas Boll wrote: 2013/8/23 Ian Romanick i...@freedesktop.org: Mesa 9.2 release candidate 2 is now available for testing. The tag in the GIT repository for Mesa 9.2-rc2 is 'mesa-9.2-rc2'. Mesa 9.2 release candidate 2 is available for download at

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Vadim Girlin
On 08/24/2013 07:24 PM, Marek Olšák wrote: See piglit/fbo-alphatest-nocolor. Ah, it seems I just compared wrong results when I was testing all combinations of backends and looked for regressions. Now I think the problem is that even though llvm backend correctly emits color export with

[Mesa-dev] [Bug 60197] Mesa Gallium VPATH build is broken

2013-08-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60197 --- Comment #6 from Quentin Sardem FF7 Glidic sardemff7+freedesk...@sardemff7.net --- Seems to be fixed in 76feef0823df75a36f264f48ec843dd863fd5b89 in a slightly different (and worse, imho) way. These Python-generated files should share a

Re: [Mesa-dev] [RFC PATCH] i965: Allow C++ type safety in the use of enum brw_urb_write_flags.

2013-08-24 Thread Francisco Jerez
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 overloads. Disadvantage (a) is trivial, not really a

Re: [Mesa-dev] [Mesa-announce] Mesa 9.2 release candidate 2

2013-08-24 Thread Ian Romanick
On 08/24/2013 09:57 AM, Sven Joachim wrote: On 2013-08-23 09:48 +0200, Andreas Boll wrote: 2013/8/23 Ian Romanick i...@freedesktop.org: Mesa 9.2 release candidate 2 is now available for testing. The tag in the GIT repository for Mesa 9.2-rc2 is 'mesa-9.2-rc2'. Mesa 9.2 release candidate 2

[Mesa-dev] [Bug 68504] 9.2-rc1 clover fails to build on ppc: cannot convert 'bool' to '__vector(4) __bool int' in return

2013-08-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68504 --- Comment #1 from Francisco Jerez curroje...@riseup.net --- bugzilla-dae...@freedesktop.org writes: Using gcc-4.7.3, llvm-3.3, clang-3.3 on Gentoo PPC [...] Causes the build to fail in clover: In file included from ./core/device.hpp:29:0,

[Mesa-dev] [PATCH 1/1] r600g: Implement the new float comparison instructions for Cayman as well.

2013-08-24 Thread Henri Verbeet
I assume this should have been part of commit 7727fbb7c5d64348994bce6682e681d6181a91e9. This (obviously) fixes a lot tests. Signed-off-by: Henri Verbeet hverb...@gmail.com --- src/gallium/drivers/r600/r600_shader.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] gen_matypes: fix cross-compiling with gcc

2013-08-24 Thread Mike Frysinger
On Thursday 18 July 2013 16:59:52 Matt Turner wrote: On Mon, Feb 4, 2013 at 6:27 PM, Mike Frysinger vap...@gentoo.org wrote: The current gen_matypes logic assumes that the host compiler will produce information that is useful for the target compiler. Unfortunately, this is not the case