https://bugs.freedesktop.org/show_bug.cgi?id=59494
Andreas Boll andreas.boll@gmail.com changed:
What|Removed |Added
QA Contact|
On 23.01.2013 03:18, Tom Stellard wrote:
On Wed, Jan 23, 2013 at 02:20:21AM +0100, Christoph Bumiller wrote:
On 23.01.2013 02:07, Vadim Girlin wrote:
On 01/23/2013 04:42 AM, Christoph Bumiller wrote:
On 23.01.2013 01:21, Vadim Girlin wrote:
On 01/23/2013 03:59 AM, Vincent Lejeune wrote:
On 23.01.2013 11:31, Christian König wrote:
On 23.01.2013 03:18, Tom Stellard wrote:
On Wed, Jan 23, 2013 at 02:20:21AM +0100, Christoph Bumiller wrote:
On 23.01.2013 02:07, Vadim Girlin wrote:
On 01/23/2013 04:42 AM, Christoph Bumiller wrote:
On 23.01.2013 01:21, Vadim Girlin wrote:
On
Looks good AFAICT.
Reviewed-by: Jose Fonseca jfons...@vmware.com
- Original Message -
---
src/glsl/s_expression.cpp |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/glsl/s_expression.cpp b/src/glsl/s_expression.cpp
index 57de9d3..52fa7ae 100644
---
On Tuesday, January 22, 2013 11:02:34 AM Eric Anholt wrote:
128 pixels of 32bpp is a tile width, and 128 pixels high of that is 16
tiles. The values I see this function having are:
mask_x = 127
mask_y = 15
draw_x = 0
draw_y = 128
image-offset = (16 * 4096)
So when we go to texture
On 01/22/2013 10:36 PM, Matt Turner wrote:
On Mon, Jan 21, 2013 at 12:49 AM, Chad Versace
chad.vers...@linux.intel.com wrote:
+ /**
+* \brief Lower a packSnorm2x16 expression.
+*
+* \param vec2_rval is packSnorm2x16's input
+* \return packSnorm2x16's output as a uint rvalue
+
On 01/22/2013 07:51 PM, Brian Paul wrote:
---
src/mesa/state_tracker/st_cb_drawpixels.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_drawpixels.c
b/src/mesa/state_tracker/st_cb_drawpixels.c
index ff8a9dc..4d9c825 100644
---
On 01/22/2013 07:51 PM, Brian Paul wrote:
---
src/glsl/s_expression.cpp |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/glsl/s_expression.cpp b/src/glsl/s_expression.cpp
index 57de9d3..52fa7ae 100644
--- a/src/glsl/s_expression.cpp
+++ b/src/glsl/s_expression.cpp
Cc-ing mesa-dev--I don't want to make this design decision in a vacuum.
For those entering the conversation, the question on the table is how to
make ARB_texture_multisample deal with the fact that on Gen7, compressed
(CMS) and non-compressed (UMS or IMS) multisampled surfaces require a
different
On 01/22/2013 05:54 PM, Carl Worth wrote:
Carl Worth cwo...@cworth.org writes:
Note: The test results mentioned here, (and in the following patches), expect
the fixes I recently submitted to the fbo-* tests in piglit. That series
consists of 5 patches starting with:
While developing this
On 01/21/2013 05:16 PM, Paul Berry wrote:
This patch paves the way for allowing varying structs by generalizing
varying_matches::compute_packing_order to handle any type of varying.
Previously, we packed in the order (vec4, vec2, float, vec3), with
matrices being packed according to the size of
On 01/23/2013 07:18 AM, Ian Romanick wrote:
On 01/22/2013 07:51 PM, Brian Paul wrote:
---
src/mesa/state_tracker/st_cb_drawpixels.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_drawpixels.c
b/src/mesa/state_tracker/st_cb_drawpixels.c
On 21 January 2013 00:49, Chad Versace chad.vers...@linux.intel.com wrote:
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_vec4.h | 3 +
src/mesa/drivers/dri/i965/brw_vec4_emit.cpp| 8 ++
On 21 January 2013 00:49, Chad Versace chad.vers...@linux.intel.com wrote:
v2: Remove lewd comment [for idr].
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
I don't have any comments on this patch beyond what Eric already said.
Hopefully I'll be able to give a pretty quick R-b once
Ian Romanick i...@freedesktop.org writes:
These tests intermittently pass / fail for me for quite some time (since
always?).
Oh, well in that case I feel better about my patch series.
We should come up with a better plan for unreliable tests in piglit. It
seems that the current approach is
On 23 January 2013 07:38, Ian Romanick i...@freedesktop.org wrote:
On 01/21/2013 05:16 PM, Paul Berry wrote:
This patch paves the way for allowing varying structs by generalizing
varying_matches::compute_**packing_order to handle any type of varying.
Previously, we packed in the order (vec4,
On Mit, 2013-01-23 at 08:17 -0800, Carl Worth wrote:
Ian Romanick i...@freedesktop.org writes:
These tests intermittently pass / fail for me for quite some time (since
always?).
Oh, well in that case I feel better about my patch series.
We should come up with a better plan for
On 01/23/2013 11:16 AM, Paul Berry wrote:
On 23 January 2013 07:38, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
On 01/21/2013 05:16 PM, Paul Berry wrote:
This patch paves the way for allowing varying structs by
generalizing
On 23 January 2013 08:36, Ian Romanick i...@freedesktop.org wrote:
On 01/23/2013 11:16 AM, Paul Berry wrote:
On 23 January 2013 07:38, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
On 01/21/2013 05:16 PM, Paul Berry wrote:
This patch paves the way for
Paul Berry stereotype...@gmail.com writes:
This patch replaces the three ir_variable_mode enums:
...
with the following five:
In my review of this patch, I'm not familiar with the code enough to
know a priori which things should become var and which should be come
function. I did look for
On 01/23/2013 07:25 AM, Ian Romanick wrote:
On 01/22/2013 07:51 PM, Brian Paul wrote:
---
src/glsl/s_expression.cpp | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/glsl/s_expression.cpp b/src/glsl/s_expression.cpp
index 57de9d3..52fa7ae 100644
---
On 23 January 2013 09:01, Carl Worth cwo...@cworth.org wrote:
Paul Berry stereotype...@gmail.com writes:
This patch replaces the three ir_variable_mode enums:
...
with the following five:
In my review of this patch, I'm not familiar with the code enough to
know a priori which things
Note, we could alternately implement this in terms of glsl_strtod()
with a (float) cast.
---
src/glsl/strtod.c | 22 ++
src/glsl/strtod.h |3 +++
2 files changed, 25 insertions(+), 0 deletions(-)
diff --git a/src/glsl/strtod.c b/src/glsl/strtod.c
index 47c1f0e..46f4dc5
Since the result of those calls is always assigned to a float.
---
src/glsl/glsl_lexer.ll| 10 +-
src/glsl/s_expression.cpp |6 +++---
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/glsl/glsl_lexer.ll b/src/glsl/glsl_lexer.ll
index 2f66c58..ddc9f80 100644
---
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
I really prefer not to see commits with only the single-line summary for
a commit message.
The single-line summary is perfect for describing what the
Ian Romanick i...@freedesktop.org writes:
From: Kenneth Graunke kenn...@whitecape.org
The existing code has a lot of duplication; the only difference between
the two cases is whether we merge in an additional layout qualifier.
I'm always in favor of a simpler grammar. Thanks!
Reviewed-by:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
For now, just drop the value on the floor.
And this gets picked up again in patch 19/32? It would be nice if the
commit messages for those two patches referenced each other a bit more
clearly, (just to make
Matt Turner matts...@gmail.com writes:
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=49360
But gen_matypes on the host will print the wrong structure offsets and
you'll build garbage code for the target.
pgpU5LTWZxOq0.pgp
Description: PGP signature
- Original Message -
Matt Turner matts...@gmail.com writes:
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=49360
But gen_matypes on the host will print the wrong structure offsets
and
you'll build garbage code for the target.
Yep. It's probably just disabling the subset of
Ian Romanick i...@freedesktop.org writes:
@@ -454,7 +460,6 @@ link_update_uniform_buffer_variables(struct gl_shader
*shader)
for (unsigned j = 0; j shader-UniformBlocks[i].NumUniforms; j++) {
if (!strcmp(var-name, shader-UniformBlocks[i].Uniforms[j].Name)) {
Abdiel Janulgue abdiel.janul...@linux.intel.com writes:
On Tuesday, January 22, 2013 11:02:34 AM Eric Anholt wrote:
128 pixels of 32bpp is a tile width, and 128 pixels high of that is 16
tiles. The values I see this function having are:
mask_x = 127
mask_y = 15
draw_x = 0
draw_y = 128
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
Misspelled recurse as recurs above.
-Carl
PS. While I'm here, the missing why part of the commit message here
might simply be:
The only difference is how to access the name.
This is one of the cases
Paul Berry stereotype...@gmail.com writes:
Cc-ing mesa-dev--I don't want to make this design decision in a vacuum.
For those entering the conversation, the question on the table is how to
make ARB_texture_multisample deal with the fact that on Gen7, compressed
(CMS) and non-compressed (UMS
Carl Worth cwo...@cworth.org writes:
Carl Worth cwo...@cworth.org writes:
Note: The test results mentioned here, (and in the following patches), expect
the fixes I recently submitted to the fbo-* tests in piglit. That series
consists of 5 patches starting with:
While developing this patch
Carl Worth cwo...@cworth.org writes:
Ian Romanick i...@freedesktop.org writes:
These tests intermittently pass / fail for me for quite some time (since
always?).
Oh, well in that case I feel better about my patch series.
We should come up with a better plan for unreliable tests in piglit.
Ian Romanick i...@freedesktop.org writes:
This is the last of the UBO instance and array instance rework for the
linker. It's a giant pile of patches, so let me explain what's going
on.
Hi Ian,
I just read through what I could of the patch series. You'll see that I
emailed out some tiny
On Wed, Jan 23, 2013 at 6:06 AM, Ian Romanick i...@freedesktop.org wrote:
On 01/22/2013 10:36 PM, Matt Turner wrote:
On Mon, Jan 21, 2013 at 12:49 AM, Chad Versace
chad.vers...@linux.intel.com wrote:
+ /**
+* \brief Lower a packSnorm2x16 expression.
+*
+* \param vec2_rval is
---
I'd like to do a new mesa-demos release.
The main motivation is the newly introduced OpenGL core profile support for
glxinfo.
CMakeLists.txt |4 ++--
configure.ac |2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/CMakeLists.txt b/CMakeLists.txt
index
The bug: The printed horizontal stride was the numerical value of the
BRW_HORIZONTAL_$N enum.
The fix: Translate the enum before printing.
Note: This is a candidate for the stable releases.
Signed-off-by: Chad Versace chad.vers...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_disasm.c | 8
This removes some bad code that got spammed into the hottest path of the
driver over time, and makes use of a few new things that didn't exist when the
driver was originally written. Results for glb21 offscreen:
x before
+ after
For our current types, the required alignment is actually just 1 byte.
When we get doubles, we have to worry (those have to be aligned to the
natural size), but we don't have doubles yet and they'll just be a
special case.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c |8 +---
1 file
---
src/mesa/drivers/dri/i965/brw_context.h |2 --
src/mesa/drivers/dri/i965/brw_draw_upload.c | 10 --
2 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h
b/src/mesa/drivers/dri/i965/brw_context.h
index f3a3efe..620f09f 100644
This was added in b93684f5f311f89c965960ab42bfea71a397b180, but there's
no need for it -- get_size has to succeed, and it has an assert for us
in debug builds.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
---
src/mesa/drivers/dri/i965/brw_draw_upload.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c
b/src/mesa/drivers/dri/i965/brw_draw_upload.c
index 43848f7..0775148 100644
--- a/src/mesa/drivers/dri/i965/brw_draw_upload.c
+++
If you want debug, set --enable-debug in your config flags.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c |2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c
b/src/mesa/drivers/dri/i965/brw_draw_upload.c
index 1c8ade5..2a3d767 100644
---
The core Mesa code has just one more case than this (GL_BITMAP), so I
don't see any cause to special-case it.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 26 ++
1 file changed, 2 insertions(+), 24 deletions(-)
diff --git
vb.inputs_read has never been a thing, even in the initial import.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c |2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c
b/src/mesa/drivers/dri/i965/brw_draw_upload.c
index 2a3d767..78ff61f 100644
---
Dereffing all the values in the two callers was just pointless, and
the function isn't inlined so there was actual code impact.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 51 ---
1 file changed, 22 insertions(+), 29 deletions(-)
diff --git
On 01/23/2013 12:35 PM, Carl Worth wrote:
Ian Romanick i...@freedesktop.org writes:
+bool
+link_uniform_blocks_are_compatible(const gl_uniform_block *a,
+ const gl_uniform_block *b)
+{
+ assert(strcmp(a-Name, b-Name) == 0);
...
+ if
On 01/23/2013 01:19 PM, Carl Worth wrote:
Ian Romanick i...@freedesktop.org writes:
@@ -454,7 +460,6 @@ link_update_uniform_buffer_variables(struct gl_shader
*shader)
for (unsigned j = 0; j shader-UniformBlocks[i].NumUniforms; j++) {
if (!strcmp(var-name,
On 01/23/2013 01:32 PM, Carl Worth wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
Misspelled recurse as recurs above.
That's what I thought, but my spell checker assured me that recurse is
wrong.
-Carl
PS. While I'm here, the missing why
Ian Romanick i...@freedesktop.org writes:
But that's functionally different.
Right, my mistake. As is then:
Reviewed-by: Carl Worth cwo...@cworth.org
-Carl
pgpCudR3n1Srw.pgp
Description: PGP signature
___
mesa-dev mailing list
v2:fix a bug with write masked inst
---
lib/Target/R600/AMDGPUTargetMachine.cpp | 2 +-
lib/Target/R600/R600LowerConstCopy.cpp | 164 +---
2 files changed, 154 insertions(+), 12 deletions(-)
diff --git a/lib/Target/R600/AMDGPUTargetMachine.cpp
https://bugs.freedesktop.org/show_bug.cgi?id=59777
Priority: medium
Bug ID: 59777
Keywords: regression
CC: bri...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp
b/lib/Target/R600/AMDILISelDAGToDAG.cpp
index 84223f6..2cc4724 100644
--- a/lib/Target/R600/AMDILISelDAGToDAG.cpp
+++
From: Vadim Girlin vadimgir...@gmail.com
Use one intrinsic for all sorts of interpolation.
Use two separate unexpanded instructions to represent INTERP_XY and _ZW -
this will allow to eliminate one part if it's not used.
Track liveness of special interpolation regs instead of reserving them -
From: Vadim Girlin vadimgir...@gmail.com
Get rid of special handling for reserved regs.
Use one intrinsic for all kinds of interpolation.
Signed-off-by: Vadim Girlin vadimgir...@gmail.com
v2[Vincent Lejeune]: Rebased against current master
---
src/gallium/drivers/r600/r600_llvm.c | 195
On 01/22/2013 07:36 PM, Matt Turner wrote:
On Mon, Jan 21, 2013 at 12:49 AM, Chad Versace
chad.vers...@linux.intel.com wrote:
Lower them to arithmetic and bit manipulation expressions.
v2:
- Rewrite using ir_builder. [for idr]
- In lowering packHalf2x16, don't truncate subnormal float16
Which LLVM version where you using? The lp_bld_init.c behavior
varies tremendously with LLVM version.
LLVM version was 3.2 System was a Ivy Bridge Quadcore (so AVX would
be the best vector ISA available on that machine) OS was Windows 7 64
Bit___
On 01/22/2013 09:19 PM, Paul Berry wrote:
On 21 January 2013 00:49, Chad Versace chad.vers...@linux.intel.com wrote:
Lower them to arithmetic and bit manipulation expressions.
v2:
- Rewrite using ir_builder. [for idr]
- In lowering packHalf2x16, don't truncate subnormal float16 values
On 01/23/2013 06:06 AM, Ian Romanick wrote:
On 01/22/2013 10:36 PM, Matt Turner wrote:
On Mon, Jan 21, 2013 at 12:49 AM, Chad Versace
chad.vers...@linux.intel.com wrote:
+ /**
+* \brief Lower a packSnorm2x16 expression.
+*
+* \param vec2_rval is packSnorm2x16's input
+*
On 23 January 2013 12:44, Chad Versace chad.vers...@linux.intel.com wrote:
On 01/22/2013 09:19 PM, Paul Berry wrote:
On 21 January 2013 00:49, Chad Versace chad.vers...@linux.intel.com
wrote:
Lower them to arithmetic and bit manipulation expressions.
v2:
- Rewrite using
On Mon, Jan 21, 2013 at 10:28:56PM +0100, Christian König wrote:
v2: don't mess up other loops
Hi Christian,
This patch regresses the glsl1-do-loop test, this test worked fine with v1 of
the patch, but the structurizer creates an infinite loop with v2. See
attached LLVM IR.
-Tom
On 01/23/2013 03:48 PM, Chad Versace wrote:
On 01/23/2013 06:06 AM, Ian Romanick wrote:
On 01/22/2013 10:36 PM, Matt Turner wrote:
On Mon, Jan 21, 2013 at 12:49 AM, Chad Versace
chad.vers...@linux.intel.com wrote:
+ /**
+* \brief Lower a packSnorm2x16 expression.
+*
+* \param
On Tue, Jan 22, 2013 at 10:19:14PM +0100, Vincent Lejeune wrote:
Reviewed-by: Tom Stellard thomas.stell...@amd.com
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 3 +++
lib/Target/R600/R600ISelLowering.cpp | 8
2 files changed, 11 insertions(+)
diff --git
On Wed, Jan 23, 2013 at 09:21:14PM +0100, Vincent Lejeune wrote:
v2:fix a bug with write masked inst
I usually only version patches if I've already submitted a previous version to
the mailiing list, so if you haven't submitted a version of this patch before
you don't need this. If this is
On Wed, Jan 23, 2013 at 09:23:03PM +0100, Vincent Lejeune wrote:
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp
b/lib/Target/R600/AMDILISelDAGToDAG.cpp
index 84223f6..2cc4724 100644
---
Carl Worth cwo...@cworth.org writes:
The renderbuffer's Format field may have an alpha channel even when the
underlying _BaseFormat does not. This can happen when mesa chooses to use
RGBA16 for an RGB16 format, for example.
So look at both fields when deciding whether to fixup the blend
Eric Anholt e...@anholt.net writes:
I think the _mesa_get_format_bits could just be dropped -- we shouldn't
have ALPHA_BITS==0 unless the baseformat has no alpha.
Good point. I did have the same thought after the fact.
Patches 3 and 4 are:
Reviewed-by: Eric Anholt e...@anholt.net
Thanks.
On Wed, Jan 23, 2013 at 09:28:34PM +0100, Vincent Lejeune wrote:
From: Vadim Girlin vadimgir...@gmail.com
Use one intrinsic for all sorts of interpolation.
Use two separate unexpanded instructions to represent INTERP_XY and _ZW -
this will allow to eliminate one part if it's not used.
Track
On Wed, Jan 23, 2013 at 09:30:02PM +0100, Vincent Lejeune wrote:
From: Vadim Girlin vadimgir...@gmail.com
Get rid of special handling for reserved regs.
Use one intrinsic for all kinds of interpolation.
Signed-off-by: Vadim Girlin vadimgir...@gmail.com
v2[Vincent Lejeune]: Rebased
On 22 January 2013 00:51, Ian Romanick i...@freedesktop.org wrote:
From: Kenneth Graunke kenn...@whitecape.org
In GLSL ES 3.00 (and GLSL 1.50), uniform blocks can have an associated
instance name, which essentially namespaces the variables inside.
This patch adds basic parsing for this new
On 22 January 2013 00:51, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
For now, just drop the value on the floor.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/ast.h | 12 ++-
src/glsl/glsl_parser.yy | 55
On 22 January 2013 00:51, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
This will soon also be used for processing interface block fields.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/ast_to_hir.cpp | 42
\On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Interfaces are structurally identical to structures from the compiler's
point of view. They have some additional restrictions, and generally
GPUs use different instructions to
Chad Versace chad.vers...@linux.intel.com writes:
The bug: The printed horizontal stride was the numerical value of the
BRW_HORIZONTAL_$N enum.
The fix: Translate the enum before printing.
Note: This is a candidate for the stable releases.
Signed-off-by: Chad Versace
Too much attention was paid to the first paragraphs, and not enough to
the last little note that oh, by the way, the rendered things
themselves still have to be clipped to just 8192 wide/high.
Fixes GTF's clip.c test with 4096 or higher width on ivb, where one of
the triangles got the upper half
Khronos has apparently decided that depth textures with sized formats
(allowed with ARB_internalformat_query or ES 3.0) should be treated as
GL_RED, while unsized formats (an existing feature) should be treated
as GL_INTENSITY for compatibility with ES 2.0.
Ian is proposing changes to
Michel Dänzer mic...@daenzer.net writes:
Are these tests unreliable themselves, or rather the driver paths
they're hitting? If it's the latter (a phenomenon which is well-known to
me with radeonsi :), neither approach seems appropriate.
Yes, that's a good question.
For
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp
b/lib/Target/R600/AMDILISelDAGToDAG.cpp
index 84223f6..b08d39f 100644
--- a/lib/Target/R600/AMDILISelDAGToDAG.cpp
+++
---
lib/Target/R600/AMDGPUTargetMachine.cpp | 2 +-
lib/Target/R600/R600LowerConstCopy.cpp | 167 +---
2 files changed, 157 insertions(+), 12 deletions(-)
diff --git a/lib/Target/R600/AMDGPUTargetMachine.cpp
b/lib/Target/R600/AMDGPUTargetMachine.cpp
index
WARNING: This patch emits VS code that violates documented hardware
restrictions and then foolishly relies on the undocumented behavior that
results from violating those restrictions.
v2: Explain undocumented hardware behavior. Improve comments.
CC: Eric Anholt e...@anholt.net
CC: Paul Berry
Chad Versace chad.vers...@linux.intel.com writes:
diff --git a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
b/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
index 324e665..9b54796 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
@@ -923,6
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
If the block has an instance name, add the instance name to the symbol
table instead of the individual fields.
Fixes the piglit test
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
For variables that are in an interface block or are an instance of an
interface block, this is the GLSL_TYPE_INTERFACE type for that block.
Convert the
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
For the first declaration below, there will be an ir_variable named
instance whose type and whose instance_type will be the same
glsl_type. For the second declaration, there will
Chad Versace chad.vers...@linux.intel.com writes:
+void
+vec4_visitor::emit_unpack_half_2x16(dst_reg dst, src_reg src0)
+{
+ if (intel-gen 7)
+ assert(!ir_unop_unpack_half_2x16 should be lowered);
+
+ assert(dst.type == BRW_REGISTER_TYPE_F);
+ assert(src0.type ==
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Not used yet, but the UBO layout visitor will use this.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/link_uniforms.cpp | 10 ++
src/glsl/linker.h
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Not used yet, but the UBO layout visitor will use this.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/link_uniforms.cpp | 15 ++-
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/ast_to_hir.cpp | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git
On 23 January 2013 11:45, Ian Romanick i...@freedesktop.org wrote:
On 01/23/2013 01:32 PM, Carl Worth wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
Misspelled recurse as recurs above.
That's what I thought, but my spell checker assured me
From: Ian Romanick ian.d.roman...@intel.com
I believe that the size used to vary, so the dynamic allocation is
necessary.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/drivers/dri/intel/intel_context.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
On 01/23/2013 10:07 PM, Paul Berry wrote:
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
mailto:ian.d.roman...@intel.com
For variables that are in an interface block or are an instance of an
On Thu, Jan 24, 2013 at 10:06 AM, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
I believe that the size used to vary, so the dynamic allocation is
necessary.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
On 01/23/2013 06:27 PM, Kenneth Graunke wrote:
Khronos has apparently decided that depth textures with sized formats
(allowed with ARB_internalformat_query or ES 3.0) should be treated as
GL_RED, while unsized formats (an existing feature) should be treated
as GL_INTENSITY for compatibility with
On 01/22/2013 03:51 AM, Ian Romanick wrote:
So here it is.
I've just pushed a re-spin of this patch series to the gles3 branch. I
have incorporated all of the review comments received so far *except*
Paul's question on patch 10, and his NULL initialization suggestion on
patch 12.
This
On 23 January 2013 21:19, Ian Romanick i...@freedesktop.org wrote:
On 01/23/2013 10:07 PM, Paul Berry wrote:
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
mailto:ian.d.romanick@intel.**com
commit 60894edeef973e86a73067276f658b72f84271b6 changed the way dri2
buffer pitch is interpreted in intel driver createImageFromName
implementation, caller must set pitch in bytes, not pixels.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/egl/drivers/dri2/platform_android.c | 2 +-
Hello;
Following patch introduces API change between Mesa and gralloc module. Gralloc
module with appropriate change is available at 01.org Android repository. This
change is not compatible with current android-x86 project gralloc. As this kind
of API changes may happen in future, I'd like to
currently a gralloc internal structure is exposed to mesa,
use a query function instead to maintain ABI compatibility.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/egl/drivers/dri2/platform_android.c | 8 ++--
src/egl/main/Android.mk
100 matches
Mail list logo