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
On 02/12/2013 08:45 PM, Eric Anholt wrote:
Tapani Pälli writes:
On 02/12/2013 12:38 AM, Eric Anholt wrote:
Tapani Pälli writes:
---
src/mapi/glapi/gen/es_EXT.xml | 13 +
src/mesa/drivers/dri/intel/intel_extensions.c | 1 +
src/mesa/main/dd.h
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
> >
> > Seems to be allot simpler, and also paves the
> > way for further improvements.
[...]
> > diff --git a/lib/Target/R600/SIInstructions.td
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
---
src/mesa/drivers/dr
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 samplerCubeArr
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 T
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 Justen
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
Cc: Ian Romanick
Cc: Matt Turner
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/intel/intel_screen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-
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
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.
Furthermor
---
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 ch
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
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,
On Tue, Feb 12, 2013 at 8:12 AM, Paul Berry 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 spec further
On Tue, Feb 12, 2013 at 06:13:22PM +0100, Christian König wrote:
> From: Christian König
>
> Seems to be allot simpler, and also paves the
> way for further improvements.
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/AMDGPU.h |1 -
> lib/Target/R600/AMD
On Tue, Feb 12, 2013 at 06:13:19PM +0100, Christian König wrote:
> From: Christian König
>
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 really these helpers and ev
Paul Berry writes:
> 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 sam
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 file
On Feb 11, 2013, at 10:39 AM, Vincent Lejeune 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 MBB.
FY
https://bugs.freedesktop.org/show_bug.cgi?id=60737
--- Comment #1 from Paul Berry ---
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 sampler types.
Mesa doe
>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 precision
On Die, 2013-02-12 at 18:13 +0100, Christian König wrote:
> From: Christian König
>
> Mark all the operands that can also have an immediate.
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstrFormats.td | 32 +-
> lib/Target/R600/SIInstructions.td | 128
> ++
This looks good. Thanks.
Reviewed-by: Marek Olšák
Marek
On Tue, Feb 12, 2013 at 12:30 PM, wrote:
> From: Jerome Glisse
>
> 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.
On 02/12/2013 11:35 AM, Jordan Justen wrote:
On Mon, Feb 11, 2013 at 4:50 PM, Anuj Phogat wrote:
On Mon, Feb 11, 2013 at 3:39 PM, Jordan Justen
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 will b
On Mon, Feb 11, 2013 at 4:50 PM, Anuj Phogat wrote:
> On Mon, Feb 11, 2013 at 3:39 PM, Jordan Justen
> 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 will be reversed.
>>
>> For now, remove
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 insertions(+
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
b/lib/Target/R600/R600ISelLowering.cpp
Tapani Pälli writes:
> On 02/12/2013 12:38 AM, Eric Anholt wrote:
>> Tapani Pälli writes:
>
>>> ---
>>> src/mapi/glapi/gen/es_EXT.xml | 13 +
>>> src/mesa/drivers/dri/intel/intel_extensions.c | 1 +
>>> src/mesa/main/dd.h| 4 ++-
>>> src
Chris Forbes 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
> - allow widt
Chris Forbes 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 always ignor
Chris Forbes 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 specialize
On Die, 2013-02-12 at 19:00 +0100, Tom Stellard wrote:
> On Tue, Feb 12, 2013 at 06:39:09PM +0100, Michel Dänzer wrote:
> > From: Michel Dänzer
> >
> > 8 more little piglits.
> >
>
> Is this a candidate for the stable branch?
Yes, thanks for the reminder.
--
Earthling Michel Dänzer
On Tue, Feb 12, 2013 at 06:39:09PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> 8 more little piglits.
>
Is this a candidate for the stable branch?
> Signed-off-by: Michel Dänzer
> ---
> src/gallium/drivers/radeonsi/radeonsi_shader.c | 29
> ++
> 1 file chan
From: Michel Dänzer
8 more little piglits.
Signed-off-by: Michel Dänzer
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 29 ++
1 file changed, 29 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_sha
From: Christian König
Signed-off-by: Christian König
---
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 b/lib/Target/R600/SIInstrInfo.td
index efc6015..8c4e5af 100644
--
From: Christian König
Using the new NearestCommonDominator class.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp |6 ++
1 file changed, 6 insertions(+)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp
b/lib/Target/R600/AMDGPUStructurizeCFG.cpp
index
From: Christian König
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 66 ++
1 file changed, 66 insertions(+)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp
b/lib/Target/R600/AMDGPUStructurizeCFG.cpp
index 169d954..ad628c1 100
From: Christian König
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
---
lib/Target/R600/AMDGPUISelLowering.h | 10 ---
lib/Target/R600/SIISelLowering.cpp | 37 -
From: Christian König
Seems to be allot simpler, and also paves the
way for further improvements.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPU.h |1 -
lib/Target/R600/AMDGPUAsmPrinter.cpp |2 -
lib/Target/R600/AMDGPUTargetMachine.cpp
From: Christian König
Mark all the operands that can also have an immediate.
Signed-off-by: Christian König
---
lib/Target/R600/SIInstrFormats.td | 32 +-
lib/Target/R600/SIInstructions.td | 128 ++---
lib/Target/R600/SIRegisterInfo.td | 10 ++-
3 f
From: Christian König
Previously it only worked because of coincident.
Signed-off-by: Christian König
---
lib/Target/R600/SIInstrFormats.td | 53 +++--
1 file changed, 16 insertions(+), 37 deletions(-)
diff --git a/lib/Target/R600/SIInstrFormats.td
b/lib/Tar
From: Christian König
Signed-off-by: Christian König
---
lib/Target/R600/SIInstrFormats.td | 66 +
lib/Target/R600/SIInstrInfo.td| 66 -
2 files changed, 66 insertions(+), 66 deletions(-)
diff --git a/lib/Target/R6
From: Christian König
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUCodeEmitter.h| 45
lib/Target/R600/MCTargetDesc/AMDGPUMCCodeEmitter.h |7 ---
2 files changed, 52 deletions(-)
delete mode 100644 lib/Target/R600/AMDGPUCodeEmitter.h
diff
From: Christian König
Generate more than one loop if it seems to make sense.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 342 +-
1 file changed, 147 insertions(+), 195 deletions(-)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG
From: Christian König
Stop adding more instructions than necessary.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 40 +-
1 file changed, 39 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp
b/lib/T
From: Christian König
Using the new NearestCommonDominator class.
Signed-off-by: Christian König
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 31 +-
1 file changed, 22 insertions(+), 9 deletions(-)
diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp
b/lib/Ta
https://bugs.freedesktop.org/show_bug.cgi?id=60686
ajax at nwnk dot net changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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.cp
From: Jerome Glisse
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 + spelling fix
Sig
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 a/lib/Target/R600/AMDILISelD
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
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 dec
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
On 02/11/2013 07:19 PM, Eric Anholt wrote:
Brian Paul 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
glLinkProgram() call
https://bugs.freedesktop.org/show_bug.cgi?id=60706
--- Comment #4 from José Fonseca ---
(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 receiving this ma
https://bugs.freedesktop.org/show_bug.cgi?id=60527
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=60524
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=60047
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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 inse
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.
On Tue, Feb 12, 2013 at 12:59:14PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
>
> Signed-off-by: Michel Dänzer
> ---
> test/CodeGen/R600/llvm.AMDGPU.tex.ll | 42 +
> test/CodeGen/R600/llvm.SI.sample.ll | 71
>
> 2 files change
https://bugs.freedesktop.org/show_bug.cgi?id=60723
Michel Dänzer changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
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
>>
>> 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
>>
On 11.02.2013 20:47, srol...@vmware.com wrote:
> From: Roland Scheidegger
>
> 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:
> http://msdn.microsoft.com/en-us/library/windows/desktop
From: Michel Dänzer
Signed-off-by: Michel Dänzer
---
test/CodeGen/R600/llvm.AMDGPU.tex.ll | 42 +
test/CodeGen/R600/llvm.SI.sample.ll | 71
2 files changed, 113 insertions(+)
create mode 100644 test/CodeGen/R600/llvm.AMDGPU.tex.ll
cre
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
http://li
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
> b/lib/Target/R600/AMDILISelDAGToDAG.cpp
> index b125ba8..2f34fe3 100
- Original Message -
> On Mon, 2013-02-11 at 20:47 +0100, srol...@vmware.com wrote:
> > From: Roland Scheidegger
> >
> > 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:
>
Good catch!
Reviewed-by: Jose Fonseca
- 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 http://bugs.freed
On 02/12/2013 12:38 AM, Eric Anholt wrote:
Tapani Pälli 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 rendering backend.
Are
70 matches
Mail list logo