Hi,
My name is Sida Li and I am a senior student from Peking University in China. I
am interested in the idea that improved application of GLSL complier
optimizations.
I have downloaded the source code and read some parts of the it. First let me
talk about my understanding about the problem.
Hello,
I am really interested in doing the GSOC 2013 project Find common patterns
in real GLSL shaders.
Implementation:
Algorithm:- Max-miner algorithm as it uses the same data structure as
Apriori i.e. hash tree.
The following implementation has been found faster than normal ways:
Max-Miner
On Sat, Apr 20, 2013 at 02:20:23PM +0200, Christian König wrote:
Am 20.04.2013 09:27, schrieb Mathias Fröhlich:
Hi Tom,
May be I need to tell where the problem really appears in real life.
OpenSceneGraph has some nifty features regarding multi channel rendering.
Assume a setup of
Hi all,
On Monday, April 22, 2013 00:39:57 Tom Stellard wrote:
[...]
The only pro for further investigating the dlopen flags is that I fear the
distribution builders who invented dynamic linking in the drivers. That change
destroyed symbol isolation in the drivers at that point. They will
From: Christopher James Halse Rogers r...@ubuntu.com
This is only exposed by drivers wich support the new PIPE_CAP_PRIME parameter,
for PRIME import/export.
---
This stubs out texture_from_renderbuffer, which I don't care about, but
that looks like it wouldn't be terribly painful to implement if
From: Christopher James Halse Rogers r...@ubuntu.com
---
src/gallium/drivers/nouveau/nouveau_screen.c | 19 ++-
src/gallium/drivers/nouveau/nouveau_screen.h | 2 ++
src/gallium/drivers/nv30/nv30_screen.c | 6 +-
src/gallium/drivers/nv50/nv50_screen.c | 5 -
From: Christopher James Halse Rogers r...@ubuntu.com
It's a map of GEM name-bo, so identify it as such
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
From: Christopher James Halse Rogers r...@ubuntu.com
---
src/gallium/drivers/r300/r300_screen.c| 8 ++-
src/gallium/drivers/r300/r300_texture.c | 2 +-
src/gallium/drivers/r600/r600_pipe.c | 7 ++-
src/gallium/drivers/r600/r600_texture.c | 2 +-
From: Christopher James Halse Rogers r...@ubuntu.com
Otherwise the default is TYPE_SHARED, which will flink the bo. This seems
unnecessary for a simple stride query.
---
src/gallium/state_trackers/dri/drm/dri2.c | 1 +
1 file changed, 1 insertion(+)
diff --git
From: Christopher James Halse Rogers r...@ubuntu.com
resource_get_handle can fail, so we might as well bubble that failure
up to the client.
---
src/gallium/state_trackers/dri/drm/dri2.c | 39 +++
1 file changed, 24 insertions(+), 15 deletions(-)
diff --git
Do not propagate a copy if source and destination are identical.
Otherwise code like
MOV TEMP[0].xyzw, TEMP[0].wzyx
mov TEMP[1].xyzw, TEMP[0].xyzw
is changed to
MOV TEMP[0].xyzw, TEMP[0].wzyx
mov TEMP[1].xyzw, TEMP[0].wzyx
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |2 ++
1 file
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/main/mtypes.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index 6108a35..7b51ec8 100644
--- a/src/mesa/main/mtypes.h
+++ b/src/mesa/main/mtypes.h
@@ -1536,8
Reviewed-by: Marek Olšák mar...@gmail.com
Marek
On Mon, Apr 22, 2013 at 8:38 AM,
christopher.halse.rog...@canonical.com wrote:
From: Christopher James Halse Rogers r...@ubuntu.com
It's a map of GEM name-bo, so identify it as such
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 16
On 21.04.2013 14:35, Jose Fonseca wrote:
- Original Message -
On 21.04.2013 13:18, Jose Fonseca wrote:
I think that drivers can just report all 4 CAPs as supported and do the
adjustment in the shader themselves (no need for recompilation, just use
uniforms, the st already does it
On Sam, 2013-04-20 at 13:21 +0200, Christian König wrote:
From: Christian König christian.koe...@amd.com
Should fix: https://bugs.freedesktop.org/show_bug.cgi?id=63702
Signed-off-by: Christian König christian.koe...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_uvd.c |6 +++---
Am 22.04.2013 09:35, schrieb Michel Dänzer:
On Sam, 2013-04-20 at 13:21 +0200, Christian König wrote:
From: Christian König christian.koe...@amd.com
Should fix: https://bugs.freedesktop.org/show_bug.cgi?id=63702
Signed-off-by: Christian König christian.koe...@amd.com
---
Am 20.04.2013 10:57, schrieb jfons...@vmware.com:
From: José Fonseca jfons...@vmware.com
They are supported on LLVM 3.1, at least on x86. (I haven't tested on PPC
though.)
Actually lp_build_linear_mip_levels() already has been emitting them for
some time.
This avoids intrinsics, which
- Original Message -
From: Christopher James Halse Rogers r...@ubuntu.com
This is only exposed by drivers wich support the new PIPE_CAP_PRIME
parameter,
for PRIME import/export.
What does exactly PIPE_CAP_PRIME entail? Please document it.
Jose
On Mon, Apr 22, 2013 at 8:38 AM,
christopher.halse.rog...@canonical.com wrote:
From: Christopher James Halse Rogers r...@ubuntu.com
---
src/gallium/drivers/r300/r300_screen.c| 8 ++-
src/gallium/drivers/r300/r300_texture.c | 2 +-
src/gallium/drivers/r600/r600_pipe.c
Am 22.04.2013 13:21, schrieb Marek Olšák:
On Mon, Apr 22, 2013 at 8:38 AM,
christopher.halse.rog...@canonical.com wrote:
From: Christopher James Halse Rogers r...@ubuntu.com
---
src/gallium/drivers/r300/r300_screen.c| 8 ++-
src/gallium/drivers/r300/r300_texture.c |
From: Alex Deucher alexander.deuc...@amd.com
There are too many cases were we end up with lockups.
Once we sort out the remaining issues on master, they
can be backported and hyperz can be re-enabled on 9.1
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
---
On 04/22/2013 12:38 AM, christopher.halse.rog...@canonical.com wrote:
From: Christopher James Halse Rogersr...@ubuntu.com
---
src/gallium/drivers/r300/r300_screen.c| 8 ++-
src/gallium/drivers/r300/r300_texture.c | 2 +-
src/gallium/drivers/r600/r600_pipe.c
On 04/21/2013 05:25 PM, Marek Olšák wrote:
Although this might be useful for ARB_clear_buffer_object,
I need it for initializating resources in r600g.
---
src/gallium/auxiliary/util/u_blitter.c | 81 +---
src/gallium/auxiliary/util/u_blitter.h | 18 ++-
2
From: José Fonseca jfons...@vmware.com
TEMP is not the only register file that accept unsigned. OUT too.
Actually, what determines the appropriate type of the destination value is
not the opcode, but rather the register.
Also cleanup/simplify code. Add a few more asserts, but also make
code
On 04/21/2013 03:09 PM, Kenneth Graunke wrote:
I was reviewing some patches and saw more new files that said IN NO EVENT
SHALL BRIAN PAUL BE LIABLE but were...not authored by Brian. People
keep doing that by accident, and a lot of those files have been altered
by other people by now anyway.
On Sun, Apr 21, 2013 at 12:30 PM, Kenneth Graunke kenn...@whitecape.org wrote:
On 04/19/2013 12:35 PM, Jordan Justen wrote:
For interface blocks, there are three separate namespaces for
uniform, input and output blocks.
http://knowyourmeme.com/photos/2109
There are?
Similarly, for your
On Mon, 2013-04-22 at 10:20 -0400, alexdeuc...@gmail.com wrote:
From: Alex Deucher alexander.deuc...@amd.com
There are too many cases were we end up with lockups.
Once we sort out the remaining issues on master, they
can be backported and hyperz can be re-enabled on 9.1
Signed-off-by:
https://bugs.freedesktop.org/show_bug.cgi?id=63404
Joe Konno joe.ko...@linux.intel.com changed:
What|Removed |Added
Depends on||54111, 59299
---
34 files changed, 311 insertions(+), 889 deletions(-)
Also, swrast_dri.so now passes two FXT1 tests that failed before, and
i915's vertex shader texturing works. I haven't tested the radeon/nouveau
code, nor have I been as invasive as I could be, because I don't have any
AGP systems left.
This is the equivalent of intel's
80513ec8b4c812b9c6249cc5824337a5f04ab34c.
---
src/mesa/drivers/dri/radeon/radeon_texture.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/radeon/radeon_texture.c
b/src/mesa/drivers/dri/radeon/radeon_texture.c
index
This function going to get used a lot more in upcoming patches.
---
src/mesa/swrast/s_texture.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/src/mesa/swrast/s_texture.c b/src/mesa/swrast/s_texture.c
index 51048be..36a90dd 100644
---
---
src/mesa/swrast/s_texture.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/mesa/swrast/s_texture.c b/src/mesa/swrast/s_texture.c
index 36a90dd..b6dd8cb 100644
--- a/src/mesa/swrast/s_texture.c
+++ b/src/mesa/swrast/s_texture.c
@@ -218,18 +218,17 @@
MapTextureImage has the exact same logic, except it can also handle
swrast-allocated buffers.
---
src/mesa/drivers/dri/nouveau/nouveau_texture.c | 99 +---
1 file changed, 18 insertions(+), 81 deletions(-)
diff --git a/src/mesa/drivers/dri/nouveau/nouveau_texture.c
---
src/mesa/drivers/dri/intel/intel_tex.c |8 +---
src/mesa/drivers/dri/radeon/radeon_texture.c |9 +
2 files changed, 2 insertions(+), 15 deletions(-)
diff --git a/src/mesa/drivers/dri/intel/intel_tex.c
b/src/mesa/drivers/dri/intel/intel_tex.c
index 2fbd5c4..24f13df
---
src/mesa/drivers/dri/intel/intel_tex.c | 20 ++--
src/mesa/drivers/dri/radeon/radeon_texture.c | 15 ++-
src/mesa/swrast/s_texture.c | 24
src/mesa/swrast/swrast.h |2 +-
4 files changed,
---
src/mesa/swrast/s_context.h |1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/swrast/s_context.h b/src/mesa/swrast/s_context.h
index 509759e..3e007b0 100644
--- a/src/mesa/swrast/s_context.h
+++ b/src/mesa/swrast/s_context.h
@@ -149,7 +149,6 @@ struct swrast_texture_image
For hardware drivers with pitch alignment requirements, a
non-power-of-two-sized texture format won't end up being an integer number
of pixels per row. Also, avoids having to change our units between
MapTextureImage's rowStride and swrast's RowStride.
This doesn't fully convert the compressed
This gets us ready for the Map field to die.
---
src/mesa/swrast/s_texfetch_tmp.h |4 ++--
src/mesa/swrast/s_texfilter.c|4 ++--
src/mesa/swrast/s_triangle.c |8
3 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/mesa/swrast/s_texfetch_tmp.h
This code is trying to deal with providing a map in the case that
AllocTexImageBuffer was called, which is hooked up to the swrast variant.
---
src/mesa/drivers/dri/nouveau/nouveau_texture.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
This is a step toward allowing drivers to use their normal mapping paths,
instead of requiring that all slice mappings come from an aligned offset
from the first slice's map.
This incidentally fixes missing slice handling in FXT1 swrast.
---
src/mesa/drivers/dri/intel/intel_tex_validate.c | 37
Now that everything goes through ImageSlices[], we can rely on the
driver's existing texture mapping function.
A big block of code goes away on Radeon that looks like it was to deal with
the validate that happened at SpanRenderStart, which no longer occurs since we
don't need validation for the
I always forget how we do this for compressed textures.
---
src/mesa/main/dd.h |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/dd.h b/src/mesa/main/dd.h
index 8f3cd3d..471d963 100644
--- a/src/mesa/main/dd.h
+++ b/src/mesa/main/dd.h
@@ -354,7 +354,9 @@
In the case of renering to windows in X, we would render to stale buffers
(or not render at all!) if you hit a MapRenderbuffer as the first thing
done to your window after new buffers are ready to be collected in DRI2.
I think this also covers the weird comment about irb-mt being missing
It's not really span code ever since we stopped using spans for S8.
---
src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 56
src/mesa/drivers/dri/intel/intel_span.c| 56
src/mesa/drivers/dri/intel/intel_span.h|2 -
3 files
Kenneth Graunke kenn...@whitecape.org writes:
The code doesn't set brw-query.obj to NULL, it sets query-bo to NULL.
Reviewed-by: Eric Anholt e...@anholt.net
pgpHt7PZ8akua.pgp
Description: PGP signature
___
mesa-dev mailing list
Marek Olšák mar...@gmail.com writes:
A driver doesn't have to set _NEW_BUFFER_OBJECT if it doesn't reallocate
the buffer from core Mesa's point of view, which avoids unnecessary state
validation.
Gallium drivers can be asked to reallocate a buffer privately by setting
Marek Olšák mar...@gmail.com writes:
already set by _mesa_readbuffer
---
src/mesa/main/buffers.c |1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/main/buffers.c b/src/mesa/main/buffers.c
index 3f5cbcd..36b0312 100644
--- a/src/mesa/main/buffers.c
+++
On Mon, Apr 22, 2013 at 11:14 AM, Eric Anholt e...@anholt.net wrote:
This function going to get used a lot more in upcoming patches.
---
src/mesa/swrast/s_texture.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/src/mesa/swrast/s_texture.c
MapBufferRange was present twice. MapBuffer was missing.
Note: This is a candidate for the stable branches.
---
src/mesa/main/context.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index ce76243..0fb57ad 100644
---
Chris Forbes chr...@ijw.co.nz writes:
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/main/mtypes.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index 6108a35..7b51ec8 100644
--- a/src/mesa/main/mtypes.h
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Tue, Apr 16, 2013 at 04:57:49PM -0700, Eric Anholt wrote:
Topi Pohjolainen topi.pohjolai...@intel.com writes:
+ if (image-regions[i]-bo)
+ intel_region_release(image-regions[i]);
Why are you looking at the BO for
On 04/22/2013 11:38 AM, Brian Paul wrote:
MapBufferRange was present twice. MapBuffer was missing.
Note: This is a candidate for the stable branches.
---
src/mesa/main/context.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/context.c
Looks like a copypaste typo.
---
src/mesa/main/context.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index ce76243..9d915b6 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -923,7 +923,6 @@
- Original Message -
Looks like a copypaste typo.
---
src/mesa/main/context.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index ce76243..9d915b6 100644
--- a/src/mesa/main/context.c
+++
On 22/04/13 19:38, Brian Paul wrote:
MapBufferRange was present twice. MapBuffer was missing.
Note: This is a candidate for the stable branches.
---
src/mesa/main/context.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/context.c
On 04/12/2013 11:21 AM, Eric Anholt wrote:
Kenneth Graunke kenn...@whitecape.org writes:
Ironlake's counters are always enabled; userspace can simply send a
MI_REPROT_PERF_COUNT packet to take a snapshot of them. This makes it
easy to implement.
The counters are documented in the source code
On 04/12/2013 01:16 PM, Eric Anholt wrote:
Kenneth Graunke kenn...@whitecape.org writes:
Ironlake's counters are always enabled; userspace can simply send a
MI_REPROT_PERF_COUNT packet to take a snapshot of them. This makes it
easy to implement.
The counters are documented in the source code
Patrick Baggett baggett.patr...@gmail.com writes:
On Mon, Apr 22, 2013 at 11:14 AM, Eric Anholt e...@anholt.net wrote:
This function going to get used a lot more in upcoming patches.
---
src/mesa/swrast/s_texture.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
On 04/13/2013 11:07 AM, Christoph Bumiller wrote:
On 12.04.2013 21:14, Kenneth Graunke wrote:
This provides an interface for applications (and OpenGL-based tools) to
access GPU performance counters. Since the exact performance counters
available vary between vendors and hardware generations,
On 04/22/2013 12:17 PM, Ferry Huberts wrote:
On 22/04/13 19:38, Brian Paul wrote:
MapBufferRange was present twice. MapBuffer was missing.
Note: This is a candidate for the stable branches.
---
src/mesa/main/context.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff
On 22/04/13 21:03, Brian Paul wrote:
On 04/22/2013 12:17 PM, Ferry Huberts wrote:
On 22/04/13 19:38, Brian Paul wrote:
MapBufferRange was present twice. MapBuffer was missing.
Note: This is a candidate for the stable branches.
---
src/mesa/main/context.c |2 +-
1 files changed,
We want to be able to reset certain parts of the pipeline,
in particular the input primitive index, but only either with
seperate invocations of the draw_vbo or new instances. In all
other cases (e.g. new invocations due to primitive restart)
that data needs to be preserved. Add a function through
Am 20.04.2013 09:04, schrieb Zack Rusin:
TGSI geometry shader input declerations are of the IN[][2] format
and the dimensions of the array have to be deduced from the input
primitive property.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 17
Can you even have that property in in a non-geometry shader?
No, not in well formed shaders at least. One could of course force it through
graw. I thought about an assert there but I figured that a conditional there
won't matter much.
In any case, series looks good to me, though your patches
---
src/glsl/builtins/tools/generate_builtins.py |1 +
src/glsl/glcpp/glcpp-parse.y |3 +++
src/glsl/glsl_parser_extras.cpp |1 +
src/glsl/glsl_parser_extras.h|2 ++
src/glsl/standalone_scaffolding.cpp |1 +
---
src/glsl/builtins/ir/bitCount.ir| 41 ++
src/glsl/builtins/ir/bitfieldExtract.ir | 57
src/glsl/builtins/ir/bitfieldInsert.ir | 65 +++
src/glsl/builtins/ir/bitfieldReverse.ir | 41
---
src/glsl/ir.cpp|8 +++-
src/glsl/ir.h | 21 -
src/glsl/ir_validate.cpp | 26 ++
src/glsl/opt_algebraic.cpp |6 +++---
This series, available at
git://people.freedesktop.org/~mattst88/mesa arb_gpu_shader5
adds support for the bit built-ins for ARB_gpu_shader5 to the GLSL
compiler and the i965 driver.
Interesting parts of the series -
- New IR operations for each built-in, plus two operations that match
i965
---
src/glsl/ir_constant_expression.cpp | 123 +++
1 files changed, 123 insertions(+), 0 deletions(-)
diff --git a/src/glsl/ir_constant_expression.cpp
b/src/glsl/ir_constant_expression.cpp
index c09e56a..119fe59 100644
--- a/src/glsl/ir_constant_expression.cpp
i965/Gen7+ and Radeon/Evergreen+ have bfm/bfi instructions to implement
bitfieldInsert() from ARB_gpu_shader5.
---
src/glsl/ir.cpp |2 +
src/glsl/ir.h | 18
src/glsl/ir_optimization.h |1 +
src/glsl/ir_validate.cpp| 12
---
src/glsl/ir_optimization.h |1 +
src/glsl/lower_instructions.cpp | 61 +++
2 files changed, 62 insertions(+), 0 deletions(-)
diff --git a/src/glsl/ir_optimization.h b/src/glsl/ir_optimization.h
index 49b1475..445dc49 100644
---
---
src/glsl/ir_optimization.h |1 +
src/glsl/lower_instructions.cpp | 82 +++
2 files changed, 83 insertions(+), 0 deletions(-)
diff --git a/src/glsl/ir_optimization.h b/src/glsl/ir_optimization.h
index 445dc49..c5405e5 100644
---
---
src/mesa/drivers/dri/i965/brw_structs.h | 31 +++
1 files changed, 19 insertions(+), 12 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_structs.h
b/src/mesa/drivers/dri/i965/brw_structs.h
index 75365e0..c322edf 100644
---
---
src/mesa/drivers/dri/i965/brw_defines.h | 11 +++
1 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
b/src/mesa/drivers/dri/i965/brw_defines.h
index 38f0356..a5d9452 100644
--- a/src/mesa/drivers/dri/i965/brw_defines.h
+++
Also update asserts to allow BFE and BFI2, which take (unsigned)
doubleword arguments.
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 34 +++---
1 files changed, 30 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
Specifically
bfe - for bitfieldExtract()
bfi1 and bfi2 - for bitfieldInsert()
bfrev - for bitfieldReverse()
cbit - for bitCount()
fbh - for findMSB()
fbl - for findLSB()
---
src/mesa/drivers/dri/i965/brw_defines.h |7 +++
src/mesa/drivers/dri/i965/brw_disasm.c |7
Don't bother scalarizing ir_binop_bfm, since its results are
identical for all channels.
---
src/mesa/drivers/dri/i965/brw_fs.cpp |7 +++
src/mesa/drivers/dri/i965/brw_fs.h |7 +++
.../dri/i965/brw_fs_channel_expressions.cpp| 37 +++
---
src/mesa/drivers/dri/i965/brw_vec4.h |7 +++
src/mesa/drivers/dri/i965/brw_vec4_emit.cpp| 29 ++
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 48
3 files changed, 84 insertions(+), 0 deletions(-)
diff --git
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 14 ++
1 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965/brw_shader.cpp
index b3bd1b9..15a0440 100644
--- a/src/mesa/drivers/dri/i965/brw_shader.cpp
+++
---
src/mesa/drivers/dri/i965/brw_disasm.c | 26 ++
1 files changed, 22 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c
b/src/mesa/drivers/dri/i965/brw_disasm.c
index 0b881b7..556e63a 100644
--- a/src/mesa/drivers/dri/i965/brw_disasm.c
This allows maintainers/packagers/testers to tag the build with
information that will be reported by GL_VERSION.
If the environemt variable or make variable MESA_VERSION_STRING_EXTRA is
set, then its values will appear in the GL_VERSION string immediately
after Mesa X.Y and before (git-xxx).
In the last hunk:
+ case ir_binop_bfm:
+ case ir_triop_bfi:
These look like leftovers?
-- Chris
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Mon, 2013-04-22 at 21:38 +1000, Dave Airlie wrote:
On Mon, Apr 22, 2013 at 9:09 PM, Jose Fonseca jfons...@vmware.com wrote:
- Original Message -
From: Christopher James Halse Rogers r...@ubuntu.com
This is only exposed by drivers wich support the new PIPE_CAP_PRIME
parameter,
+ case ir_unop_find_msb:
+ for (unsigned c = 0; c components; c++) {
+ int v = op[0]-value.i[c];
+
+ if (v == 0 || (op[0]-type-base_type == GLSL_TYPE_INT v == -1))
+data.i[c] = -1;
+ else {
+int count = 0;
+int top_bit =
Scratch that -- I see what they're for. They seem to be missing from
enum ir_expression_operation though.
-- Chris
On Tue, Apr 23, 2013 at 1:33 PM, Chris Forbes chr...@ijw.co.nz wrote:
In the last hunk:
+ case ir_binop_bfm:
+ case ir_triop_bfi:
These look like leftovers?
-- Chris
https://bugs.freedesktop.org/show_bug.cgi?id=62142
Gordon Jin gordon@intel.com changed:
What|Removed |Added
Status|RESOLVED|REOPENED
From: Tom Stellard thomas.stell...@amd.com
The state tracker should be responsible for waiting for the kernel to
finish.
---
src/gallium/drivers/r600/evergreen_compute.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/src/gallium/drivers/r600/evergreen_compute.c
From: Tom Stellard thomas.stell...@amd.com
---
src/gallium/drivers/r600/evergreen_compute.c | 47 +++---
.../drivers/r600/evergreen_compute_internal.h | 2 +-
2 files changed, 33 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/r600/evergreen_compute.c
From: Tom Stellard thomas.stell...@amd.com
There was a lot of code in evergreen_compute_internal.c that was not
being used at all and most of it was duplicating code from other parts
of the driver.
---
src/gallium/drivers/r600/Makefile.sources | 1 -
---
Do these comments make sense to anyone else?
src/mesa/drivers/dri/i965/brw_defines.h |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
b/src/mesa/drivers/dri/i965/brw_defines.h
index ea5d2af..6517f24 100644
---
On Mon, 22 Apr, 2013 at 4:38 PM, christopher.halse.rog...@canonical.com
wrote:
From: Christopher James Halse Rogers r...@ubuntu.com
This is only exposed by drivers wich support the new PIPE_CAP_PRIME
parameter,
for PRIME import/export.
---
This stubs out texture_from_renderbuffer, which I
91 matches
Mail list logo