On 02/12/2013 12:38 AM, Eric Anholt wrote:
Tapani Pälli tapani.pa...@intel.com writes:
This patch implements a stub for GL_EXT_discard_framebuffer with
required checks listed by the extension specification. This extension
is required by GLBenchmark 2.5 when compiled with OpenGL ES 2.0
as the
Good catch!
Reviewed-by: Jose Fonseca jfons...@vmware.com
- Original Message -
For PIPE_FORMAT_Z24_UNORM_S8_UINT, the Z bits are in the 24
least significant bits.
Fixes http://bugs.freedesktop.org/show_bug.cgi?id=60527
and http://bugs.freedesktop.org/show_bug.cgi?id=60524
and
- Original Message -
On Mon, 2013-02-11 at 20:47 +0100, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
It looks like using coord.w as explicit lod value is a mistake, most likely
because some dx10 docs had it specified that way. Seems this was changed
Hi,
When we initialize OpenGL using SDL do we need to take care of the rendering
part or does SDL does the output rendering and just uses the OpenGL functions?
Thanks and Regards
Ritvik
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
test/CodeGen/R600/llvm.AMDGPU.tex.ll | 42 +
test/CodeGen/R600/llvm.SI.sample.ll | 71
2 files changed, 113 insertions(+)
create mode
On 11.02.2013 20:47, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
It looks like using coord.w as explicit lod value is a mistake, most likely
because some dx10 docs had it specified that way. Seems this was changed
though:
Am 12.02.2013 08:06, schrieb Michel Dänzer:
On Mon, 2013-02-11 at 20:47 +0100, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
It looks like using coord.w as explicit lod value is a mistake, most likely
because some dx10 docs had it specified that way. Seems this was
https://bugs.freedesktop.org/show_bug.cgi?id=60723
Michel Dänzer mic...@daenzer.net changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop
On Tue, Feb 12, 2013 at 12:59:14PM +0100, Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
test/CodeGen/R600/llvm.AMDGPU.tex.ll | 42 +
test/CodeGen/R600/llvm.SI.sample.ll | 71
On Tue, Feb 12, 2013 at 10:24:24AM +0100, Michel Dänzer wrote:
On Mon, 2013-02-11 at 19:39 +0100, Vincent Lejeune wrote:
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 24
1 file changed, 24 insertions(+)
diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp
On Die, 2013-02-12 at 15:26 +0100, Tom Stellard wrote:
On Tue, Feb 12, 2013 at 10:24:24AM +0100, Michel Dänzer wrote:
On Mon, 2013-02-11 at 19:39 +0100, Vincent Lejeune wrote:
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 24
1 file changed, 24 insertions(+)
https://bugs.freedesktop.org/show_bug.cgi?id=60047
Brian Paul bri...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=60524
Brian Paul bri...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=60527
Brian Paul bri...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=60706
--- Comment #4 from José Fonseca jfons...@vmware.com ---
(In reply to comment #3)
This regression is llvm-3.3 specific. The test passes with llvm-3.2 and
llvm-3.1.
I've reported the regression to LLVM Bugzilla.
Thanks Vinson!
--
You are
On 02/11/2013 07:19 PM, Eric Anholt wrote:
Brian Paulbri...@vmware.com writes:
Google Earth v7 calls glReleaseShaderCompiler() and Mesa's GLSL
compiler is crashing.
In particular, it looks like glReleaseShaderCompiler() is getting
called between glCompileShader() and glLinkProgram(). The
In the GLSL 1.30 spec, section 4.3.6 (Outputs) says:
If a vertex output is a signed or unsigned integer or integer
vector, then it must be qualified with the interpolation qualifier
flat.
The GLSL ES 3.00 spec further clarifies, in section 4.3.6 (Output
Variables):
Vertex shader
All of the GLSL specs from GLSL 1.30 (and GLSL ES 3.00) onward contain
language requiring certain integer variables to be declared with the
flat keyword, but they differ in exactly *when* the rule is
enforced:
(a) GLSL 1.30 and 1.40 say that vertex shader outputs having integral
type must be
https://bugs.freedesktop.org/show_bug.cgi?id=60737
Priority: medium
Bug ID: 60737
Assignee: mesa-dev@lists.freedesktop.org
Summary: In GLSL ES, a missing FS precision qualifier does not
generate an error
Severity: normal
It fixes around 100 tfb piglit tests and 16 glean tests.
NOTE: This is a candidate for the Mesa stable branch.
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 2 ++
lib/Target/R600/R600LowerConstCopy.cpp | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git
From: Jerome Glisse jgli...@redhat.com
Seems that alpha test being enabled confuse the GPU on the order in
which it should perform the Z testing. So force the order programmed
throught db shader control.
v2: Only force z order when alpha test is enabled
v3: Update db shader when binding new dsa
On Tue, Feb 12, 2013 at 05:23:43PM +0100, Vincent Lejeune wrote:
It fixes around 100 tfb piglit tests and 16 glean tests.
NOTE: This is a candidate for the Mesa stable branch.
Hi Vincent,
Could you add a lit test for this as well?
Thanks,
Tom
---
lib/Target/R600/AMDILISelDAGToDAG.cpp |
https://bugs.freedesktop.org/show_bug.cgi?id=60686
ajax at nwnk dot net a...@nwnk.net changed:
What|Removed |Added
Status|NEW |RESOLVED
From: Christian König christian.koe...@amd.com
Using the new NearestCommonDominator class.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 31 +-
1 file changed, 22 insertions(+), 9 deletions(-)
diff --git
From: Christian König christian.koe...@amd.com
Stop adding more instructions than necessary.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 40 +-
1 file changed, 39 insertions(+), 1 deletion(-)
diff --git
From: Christian König christian.koe...@amd.com
Generate more than one loop if it seems to make sense.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 342 +-
1 file changed, 147 insertions(+), 195 deletions(-)
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUCodeEmitter.h| 45
lib/Target/R600/MCTargetDesc/AMDGPUMCCodeEmitter.h |7 ---
2 files changed, 52 deletions(-)
delete mode
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 66 +
lib/Target/R600/SIInstrInfo.td| 66 -
2 files changed, 66
From: Christian König christian.koe...@amd.com
Previously it only worked because of coincident.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 53 +++--
1 file changed, 16 insertions(+), 37 deletions(-)
diff
From: Christian König christian.koe...@amd.com
Mark all the operands that can also have an immediate.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 32 +-
lib/Target/R600/SIInstructions.td | 128 ++---
From: Christian König christian.koe...@amd.com
Seems to be allot simpler, and also paves the
way for further improvements.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPU.h |1 -
lib/Target/R600/AMDGPUAsmPrinter.cpp |
From: Christian König christian.koe...@amd.com
It's completely unnecessary and can be replace with proper
SReg_64 handling instead.
This actually fixes a piglit test on SI.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUISelLowering.h | 10 ---
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 66 ++
1 file changed, 66 insertions(+)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp
From: Christian König christian.koe...@amd.com
Using the new NearestCommonDominator class.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp |6 ++
1 file changed, 6 insertions(+)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td|4
lib/Target/R600/SIInstructions.td |5 +
2 files changed, 9 insertions(+)
diff --git a/lib/Target/R600/SIInstrInfo.td
From: Michel Dänzer michel.daen...@amd.com
8 more little piglits.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 29 ++
1 file changed, 29 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
On Tue, Feb 12, 2013 at 06:39:09PM +0100, Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
8 more little piglits.
Is this a candidate for the stable branch?
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 29
Chris Forbes chr...@ijw.co.nz writes:
On Gen6, lower this to `ld` with lod=0 and an extra sample_index
parameter.
On Gen7, use `ld2dms`. We don't support CMS yet for multisample
textures, so we just hardcode MCS=0. This is ignored for IMS and UMS
surfaces.
Note: If we do end up emitting
Chris Forbes chr...@ijw.co.nz writes:
On Gen6, lower this to `ld` with lod=0 and an extra sample_index
parameter.
On Gen7, use `ld2dms`. This takes an additional MCS parameter to support
compressed multisample surfaces, but we're not enabling them for
multisample textures for now, so it's
Chris Forbes chr...@ijw.co.nz writes:
V2: - fix formatting issues
- generate GL_OUT_OF_MEMORY if teximage cannot be allocated
- fix for state moving from texobj to image
V3: - remove ridiculous stencil hack
- alter format check to not allow a base format of STENCIL_INDEX
-
Tapani Pälli tapani.pa...@intel.com writes:
On 02/12/2013 12:38 AM, Eric Anholt wrote:
Tapani Pälli tapani.pa...@intel.com writes:
---
src/mapi/glapi/gen/es_EXT.xml | 13 +
src/mesa/drivers/dri/intel/intel_extensions.c | 1 +
src/mesa/main/dd.h
NOTE: This is a candidate for the Mesa stable branch.
---
lib/Target/R600/R600ISelLowering.cpp | 6 --
lib/Target/R600/R600Instructions.td | 9 -
2 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/lib/Target/R600/R600ISelLowering.cpp
NOTE: This is a candidate for the Mesa stable branch.
---
src/gallium/drivers/r600/r600_llvm.c| 10 +++---
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 17 +
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 4 ++--
3 files changed, 26
On Mon, Feb 11, 2013 at 4:50 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
On Mon, Feb 11, 2013 at 3:39 PM, Jordan Justen
jordan.l.jus...@intel.com wrote:
In OpenGL 4.3, new language was added that would require
this check. But, if this check results in broken applications
then perhaps it
On 02/12/2013 11:35 AM, Jordan Justen wrote:
On Mon, Feb 11, 2013 at 4:50 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
On Mon, Feb 11, 2013 at 3:39 PM, Jordan Justen
jordan.l.jus...@intel.com wrote:
In OpenGL 4.3, new language was added that would require
this check. But, if this check
This looks good. Thanks.
Reviewed-by: Marek Olšák mar...@gmail.com
Marek
On Tue, Feb 12, 2013 at 12:30 PM, j.gli...@gmail.com wrote:
From: Jerome Glisse jgli...@redhat.com
Seems that alpha test being enabled confuse the GPU on the order in
which it should perform the Z testing. So force
On Die, 2013-02-12 at 18:13 +0100, Christian König wrote:
From: Christian König christian.koe...@amd.com
Mark all the operands that can also have an immediate.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 32 +-
From GLSL ES 3.00 section 4.5.4 (Default Precision Qualifiers):
The precision statement
precision precision-qualifier type;
can be used to establish a default precision qualifier. The type
field can be either int or float or any of the sampler types, and
the
https://bugs.freedesktop.org/show_bug.cgi?id=60737
--- Comment #1 from Paul Berry stereotype...@gmail.com ---
Ok, this turned out to be more complex than I thought (don't they always?). In
addition to the problem described above:
1. The spec says that default precision statements are alowed on
On Feb 11, 2013, at 10:39 AM, Vincent Lejeune v...@ovi.com wrote:
+// Using LiveInterval should make things a lot more efficient, but we
+// can't access them inside a MachineSchedStrategy.
+// Scheduling occurs on a per MBB basis, so it is sufficient to get deps
+// inside a
It fixes around 100 tfb piglit tests and 16 glean tests.
NOTE: This is a candidate for the Mesa stable branch.
---
lib/Target/R600/AMDILISelDAGToDAG.cpp | 2 ++
lib/Target/R600/R600LowerConstCopy.cpp | 2 +-
test/CodeGen/R600/kcache-fold.ll | 52 ++
3
On Tue, Feb 12, 2013 at 06:13:19PM +0100, Christian König wrote:
From: Christian König christian.koe...@amd.com
SIInstrFormats.td should contain the instruction encoding definitions
and everything else should go in SIInstrInfo.td. I got this backwards,
when I first created these files, so
On Tue, Feb 12, 2013 at 06:13:22PM +0100, Christian König wrote:
From: Christian König christian.koe...@amd.com
Seems to be allot simpler, and also paves the
way for further improvements.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPU.h
On Tue, Feb 12, 2013 at 8:12 AM, Paul Berry stereotype...@gmail.com wrote:
In the GLSL 1.30 spec, section 4.3.6 (Outputs) says:
If a vertex output is a signed or unsigned integer or integer
vector, then it must be qualified with the interpolation qualifier
flat.
The GLSL ES 3.00
I should say fix, but it has never been used until now.
S8Z24 is the format equivalent to the GL_UNSIGNED_INT_24_8 packing,
so we'll start to see it more often with st/mesa now making smart decisions
about formats.
The DB-CB copy can change the channel ordering for transfers, other than
that, the
Broken by 624528834f53f54c7a934f929769b7e6b230a0b1.
---
src/mesa/state_tracker/st_atom_texture.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_texture.c
b/src/mesa/state_tracker/st_atom_texture.c
index 28327bc..fc2d690 100644
---
src/mesa/state_tracker/st_cb_drawpixels.c | 28 +++
src/mesa/state_tracker/st_cb_texture.c| 30 ++---
src/mesa/state_tracker/st_format.c| 35 +
src/mesa/state_tracker/st_format.h|3 +++
4 files
Normally, SEND messages read existing MRF values, which were previously
written by some other instruction. However, some meta-instructions
(like FS_OPCODE_UNIFORM_PULL_CONSTANT_LOAD) implicitly write MRFs before
reading them. In that case, the existing MRF values aren't actually
read.
Meta-instructions that implicitly write then read/consume a MRF value
don't pose write-after-write conflicts with each other, since they're
actually:
- Write value 1, then consume it.
- Write value 2, then consume it.
These can safely be done in either order.
By tracking the last implicit write
We've fixed all the conformance failures, so we can turn it on now.
NOTE: This is a candidate for the 9.1 branch.
Cc: Eric Anholt e...@anholt.net
Cc: Ian Romanick i...@freedesktop.org
Cc: Matt Turner matts...@gmail.com
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
Previously when an input varying was optimized out of the
FS we would still retain it as an output of the VS.
We now build a hash of live FS input varyings rather
than looking in the FS symbol table. (The FS symbol table
will still contain the optimized out varyings.)
Signed-off-by: Jordan
Paul, Ken,
In case you were interested, I confirmed that this fixes (or more
accurately it masks) the Steam Big Picture issue that Ken fixed in
09fbc298. (I checked the effects of this patch with 09fbc298 removed.)
You two had suspected this change might mask that particular issue.
-Jordan
On
GLSL provides gradients for the 'r' coordinate (face ID), while our
hardware apparently ignores them. Sadly, this means that sample_d and
sample_d_c appear to be unsuitable for OpenGL, and need to be lowered.
For now, only handle samplerCubeShadow; we need tests for samplerCube
and
The shadow comparitor needs to be loaded into the Z component of the
last DWord.
Fixes es3conform's shadow_execution_vert and oglconform's
shadow-grad advanced.textureGrad.1D tests on Haswell.
NOTE: This is a candidate for stable branches.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
On Die, 2013-02-12 at 19:39 -0500, Tom Stellard wrote:
On Tue, Feb 12, 2013 at 06:13:22PM +0100, Christian König wrote:
From: Christian König christian.koe...@amd.com
Seems to be allot simpler, and also paves the
way for further improvements.
[...]
diff --git
On 02/12/2013 08:45 PM, Eric Anholt wrote:
Tapani Pälli tapani.pa...@intel.com writes:
On 02/12/2013 12:38 AM, Eric Anholt wrote:
Tapani Pälli tapani.pa...@intel.com writes:
---
src/mapi/glapi/gen/es_EXT.xml | 13 +
src/mesa/drivers/dri/intel/intel_extensions.c |
On 02/08/2013 01:19 AM, Chris Forbes wrote:
V2: - emit `sample` parameter properly for multisample texelFetch()
- fix spurious whitespace change
- introduce a new opcode ir_txf_ms rather than overloading the
existing ir_txf further. This makes doing the right thing in
the
67 matches
Mail list logo