MinGW has snprintf.
The patch fixes these warnings with the MinGW SCons build.
src/gallium/auxiliary/util/u_snprintf.c:459:1: warning: no previous prototype
for ‘util_vsnprintf’ [-Wmissing-prototypes]
src/gallium/auxiliary/util/u_snprintf.c:1436:1: warning: no previous prototype
for ‘util_snpri
In addition to registers used by instructions, fs_visitor maintains
direct references to certain "special" values used for inputs/outputs.
When I added VGRF compaction, I overlooked these, believing that these
direct references weren't used once instructions were generated. That
was wrong. For e
and allow thread offloading for dri2SwapBuffers.
The extension takes care of throttling, drawable flushing and
GL context flushing (no need to do glFlush), and allows thread offloading
xcb_dri2_swap_buffers. The driver has to implement the offloading, libGL
only gives it the pointer to the swap fu
---
src/glx/dri2_glx.c |6 +-
src/glx/dri_glx.c |6 +-
src/glx/drisw_glx.c |7 ++-
src/glx/glxclient.h |2 +-
src/glx/glxcmds.c |8 +++-
5 files changed, 20 insertions(+), 9 deletions(-)
diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c
index 12b3026..f
Hi,
this series adds a new DRI2 extension to the libGL-Mesa interface, which
consolidates the 3 flushes (glFlush, drawable flush, throttling) into one flush
call like my previous patch series did, but it also allows thread offloading of
SwapBuffers.
The motivation for the thread offloading is
There's no point: opt_algebraic() doesn't use any liveness information.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index dc2e5a6..f25fd16
On Wed, Nov 14, 2012 at 4:45 PM, Kenneth Graunke wrote:
> On 11/14/2012 10:34 AM, Matt Turner wrote:
>>
>> Fixes 54 gles3conform tests.
>> ---
>> src/glsl/glcpp/glcpp-parse.y | 59
>> +
>> 1 files changed, 30 insertions(+), 29 deletions(-)
>>
>> The cha
On 11/14/2012 10:34 AM, Matt Turner wrote:
Fixes 54 gles3conform tests.
---
src/glsl/glcpp/glcpp-parse.y | 59 +
1 files changed, 30 insertions(+), 29 deletions(-)
The changes in whitespace make this patch hard to read. Here it is
without whitespace c
On 11/14/2012 03:56 PM, Eric Anholt wrote:
Fixes piglit "vertex-program-two-side enabled front back" and 4 others
on my ironlake system.
---
This is a slight variation on a hunk in Olivier Galibert's series,
adding in the required state flags.
src/mesa/drivers/dri/i965/brw_sf.c |7 ---
brw_optimize.c's brw_opcodes table was a copy of brw_disasm.c's
opcode_descs table, but with an additional field: is_arith. Now that
I've deleted that, the two are identical. Keep the one in brw_disasm.c.
---
src/mesa/drivers/dri/i965/brw_context.h | 7
src/mesa/drivers/dri/i965/brw_fs
Fixes piglit "vertex-program-two-side enabled front back" and 4 others
on my ironlake system.
---
This is a slight variation on a hunk in Olivier Galibert's series,
adding in the required state flags.
src/mesa/drivers/dri/i965/brw_sf.c |7 ---
1 file changed, 4 insertions(+), 3 deletions
On Wed, Nov 14, 2012 at 1:49 PM, Andreas Boll
wrote:
> fixes errors ./configure was complaining about
>
> NOTE: This is a candidate for the 9.0 branch.
>
> Cc: Matt Turner
> ---
> If you want I can squash all three patches before pushing.
> Only tested on 9.0 branch.
>
> Makefile.am |2 ++
>
fixes errors ./configure and make was complaining about
NOTE: This is a candidate for the 9.0 branch.
Cc: Matt Turner
---
Makefile.am |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 96ab449..a5708a3 100644
--- a/Makefile.am
+++ b/Mak
fixes errors ./configure was complaining about
NOTE: This is a candidate for the 9.0 branch.
Cc: Matt Turner
---
Makefile.am | 51 ++-
1 files changed, 50 insertions(+), 1 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 7f32bc2..96ab4
fixes errors ./configure was complaining about
NOTE: This is a candidate for the 9.0 branch.
Cc: Matt Turner
---
If you want I can squash all three patches before pushing.
Only tested on 9.0 branch.
Makefile.am |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/Makefile.
I think this just hides a bug elsewhere. If a colorbuffer format is
unsupported, it should never appear in set_framebuffer_state. We
should add assert(format != ~0U) everywhere we use the return value of
si_translate_colorformat and fix any issue that leads to an assertion failure.
Marek
On Wed,
https://bugs.freedesktop.org/show_bug.cgi?id=52034
--- Comment #4 from Pavel Ondračka ---
(In reply to comment #3)
> Piglit depthstencil-default_fb-blit passes for me on llvmpipe, softpipe and
> r600g
>
> mesa: ca5840afb0eae56f86668d0b5562c6d82fe8ad4b
>
> @Vinson, Pavel: Could you confirm that
On 14.11.2012 20:14, alexdeuc...@gmail.com wrote:
From: Alex Deucher
setting the DB format to Z_INVALID disables the depth buffer.
Signed-off-by: Alex Deucher
For this series:
Reviewed-by: Christian König
---
src/gallium/drivers/radeonsi/si_state.c | 10 +++---
1 files changed,
Signed-off-by: Dmitry Cherkassov
---
src/gallium/include/pipe/p_context.h |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/gallium/include/pipe/p_context.h
b/src/gallium/include/pipe/p_context.h
index 88e12b2..c5dcae5 100644
--- a/src/gallium/include/pipe/p_context
On 11/14/2012 10:34 AM, Matt Turner wrote:
Fixes 54 gles3conform tests.
---
src/glsl/glcpp/glcpp-parse.y | 59 +
1 files changed, 30 insertions(+), 29 deletions(-)
The changes in whitespace make this patch hard to read. Here it is
without whitespace c
Reviewed-by: Ian Romanick
On 11/09/2012 02:19 AM, Andreas Boll wrote:
---
Note: For the 9.0 branch only
bin/.cherry-ignore | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
create mode 100644 bin/.cherry-ignore
diff --git a/bin/.cherry-ignore b/bin/.cherry-ignore
new
This series is
Reviewed-by: Ian Romanick
On 11/09/2012 02:18 AM, Andreas Boll wrote:
From: Ian Romanick
---
Note: For the 8.0 branch only
bin/.cherry-ignore | 68
1 files changed, 68 insertions(+), 0 deletions(-)
create mode 10064
https://bugs.freedesktop.org/show_bug.cgi?id=57127
Priority: high
Bug ID: 57127
CC: matts...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: "make dist" fails with "No rule to make target
`glapi_gentable.c',
From: Alex Deucher
Clean up a few magic numbers and rework the code a bit.
Signed-off-by: Alex Deucher
---
src/gallium/drivers/radeonsi/si_state.c | 22 ++
src/gallium/drivers/radeonsi/sid.h |2 ++
2 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/
From: Alex Deucher
setting the CB format to Z_INVALID disables the color buffer.
Signed-off-by: Alex Deucher
---
src/gallium/drivers/radeonsi/si_state.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/ra
From: Alex Deucher
setting the DB format to Z_INVALID disables the depth buffer.
Signed-off-by: Alex Deucher
---
src/gallium/drivers/radeonsi/si_state.c | 10 +++---
1 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/driv
We have a special case where non-shadow comparison with LOD requires using a
SIMD16 vec4 in an 8-wide shader, which appears in the register allocator as a
size 8 vgrf.
Fixes assertions in various piglit tests and webgl conformance.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=56521
---
On Wed, Nov 14, 2012 at 10:34 AM, Matt Turner wrote:
> - if (extensions->OES_EGL_image_external)
> - add_builtin_define(parser, "GL_OES_EGL_image_external", 1);
> + if (extensions->OES_EGL_image_external)
> +add_builtin_define(parser, "GL_OES_EGL_im
https://bugs.freedesktop.org/show_bug.cgi?id=51531
Andreas Boll changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #2 from Andreas Boll
In _mesa_initialize_context:
* Call Driver.InitializeVersionFeatures to allow the driver to
enable features which are significant to computing the versions
* Call _mesa_compute_version. It is important that this is done
before _mesa_create_exec_table is called.
In _mesa_make_current:
* Assert
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_context.c| 23 ++-
src/mesa/drivers/dri/intel/intel_context.c | 18 +-
src/mesa/drivers/dri/intel/intel_context.h |3 +++
3 files changed, 34 insertions(+), 10 deletions(-)
mode change
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/swrast/swrast.c | 74 +++---
1 file changed, 41 insertions(+), 33 deletions(-)
diff --git a/src/mesa/drivers/dri/swrast/swrast.c
b/src/mesa/drivers/dri/swrast/swrast.c
index 45ef3c2..b2f2a34 100644
--- a/src/mes
During context initialization, this callback allows the driver
to configure features of the context so the correct GL/GLES
version will be calculated.
Signed-off-by: Jordan Justen
---
src/mesa/drivers/common/driverfuncs.c |1 +
src/mesa/main/dd.h|6 ++
2 files cha
Within _mesa_initialize_context the call to _mesa_create_exec_table
now depends on ctx->Version being set. (For GLES3 support.)
This series adds a driver callback function that allows each driver
to enable features that are significant for computing to version.
Now _mesa_initialize_context will:
Fixes 54 gles3conform tests.
---
src/glsl/glcpp/glcpp-parse.y | 59 +
1 files changed, 30 insertions(+), 29 deletions(-)
The changes in whitespace make this patch hard to read. Here it is
without whitespace changes:
--- a/src/glsl/glcpp/glcpp-parse.y
+++
https://bugs.freedesktop.org/show_bug.cgi?id=52034
Andreas Boll changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #3 from Andreas Boll
https://bugs.freedesktop.org/show_bug.cgi?id=57121
--- Comment #2 from Tomasz Kaźmierczak ---
Created attachment 70080
--> https://bugs.freedesktop.org/attachment.cgi?id=70080&action=edit
The actual result (the bug)
--
You are receiving this mail because:
You are the assignee for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=57121
--- Comment #1 from Tomasz Kaźmierczak ---
Created attachment 70079
--> https://bugs.freedesktop.org/attachment.cgi?id=70079&action=edit
The expected result
--
You are receiving this mail because:
You are the assignee for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=57121
Priority: medium
Bug ID: 57121
Assignee: mesa-dev@lists.freedesktop.org
Summary: corrupted GLSL built-in function results when using
Uniform Buffer contents as arguments
Seve
On Mit, 2012-11-14 at 15:54 +0100, Christian König wrote:
> Based on already merged v4 of "fix SGPR liveness"
This change causes the following piglit tests to crash for me:
shaders/glsl-fs-atan-2
shaders/glsl-fs-sqrt-branch
spec/glsl-1.10/execution/built-in-functions/fs-atan-vec2-vec2
spec/glsl-
On Nov 13, 2012 11:09 PM, "Ross Burton" wrote:
>
> The argument --without-glut is transformed to --with-glut=no, which the
logic
> wasn't handling at all so --without-glut didn't work. Rewrite the logic
to
> handle the case where the value passed to --with-glut is "no".
>
> Signed-off-by: Ross Bu
On 11/13/2012 04:22 PM, Eric Anholt wrote:
The goal of that change was to skip counting things that aren't actually
outputs from the VS to the FS. However, explicit_location isn't set in
the case of linker-assigned locations (the common case), so basically
varying component counting got disabled
On 11/14/2012 12:16 AM, Vinson Lee wrote:
MinGW has snprintf.
The patch fixes these warnings with the MinGW SCons build.
src/gallium/auxiliary/util/u_snprintf.c:459:1: warning: no previous prototype
for ‘util_vsnprintf’ [-Wmissing-prototypes]
src/gallium/auxiliary/util/u_snprintf.c:1436:1: war
Based on already merged v4 of "fix SGPR liveness"
Signed-off-by: Christian König
---
lib/Target/AMDGPU/AMDGPU.h|4 +-
lib/Target/AMDGPU/AMDGPUTargetMachine.cpp | 12 ++-
lib/Target/AMDGPU/SIFixSGPRLiveness.cpp | 152 +++--
3 files changed, 89 inse
On 11/13/2012 05:02 PM, Kenneth Graunke wrote:
On 11/13/2012 01:44 PM, Maxence Le Doré wrote:
[snip]
diff --git a/src/glsl/ir.h b/src/glsl/ir.h
index 89c516c..5d5b56d 100644
--- a/src/glsl/ir.h
+++ b/src/glsl/ir.h
@@ -1334,7 +1334,8 @@ enum ir_texture_opcode {
ir_txl, /**< Texture look-up with e
On Wed, Nov 14, 2012 at 6:39 AM, Michel Dänzer wrote:
> From: Michel Dänzer
>
>
> Signed-off-by: Michel Dänzer
Reviewed-by: Alex Deucher
> ---
> lib/Target/AMDGPU/SIInstructions.td |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/Target/AMDGPU/SIInstruction
On Mit, 2012-11-14 at 14:18 +0100, Christian König wrote:
> SGPRs are not affected by control flow.
>
> v2: don't use liveness analyse any more, handle partial
> dominated end notes.
> v3: fix old pass name in CMakeLists.txt
> v4: remove unnecessary successor handling and kill handling
> v5:
SGPRs are not affected by control flow.
v2: don't use liveness analyse any more, handle partial
dominated end notes.
v3: fix old pass name in CMakeLists.txt
v4: remove unnecessary successor handling and kill handling
v5: adjust live analysis directly
Signed-off-by: Christian König
---
lib/T
From: Michel Dänzer
Signed-off-by: Michel Dänzer
---
lib/Target/AMDGPU/SIInstructions.td |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/Target/AMDGPU/SIInstructions.td
b/lib/Target/AMDGPU/SIInstructions.td
index 8512b48..97d2faa 100644
--- a/lib/Target/AMDGPU/S
From: Michel Dänzer
Signed-off-by: Michel Dänzer
---
lib/Target/AMDGPU/CMakeLists.txt |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/Target/AMDGPU/CMakeLists.txt b/lib/Target/AMDGPU/CMakeLists.txt
index 2a1f052..b6b3417 100644
--- a/lib/Target/AMDGPU/CMakeLists.txt
From: Michel Dänzer
Pointed out by compiler warning:
/home/daenzer/src/llvm-git/llvm/lib/Target/AMDGPU/InstPrinter/AMDGPUInstPrinter.cpp:96:16:
warning: adding 'int64_t' (aka 'long') to a string does not append to the
string [-Wstring-plus-int]
O << " + " + Op.getImm();
~~^~~~
Resending (parts of) some patches which seem to have been missed.
[PATCH RESEND 1/3] AMDGPU: Fix string concatenation in
[PATCH RESEND 2/3] AMDGPU: Don't allow using SI SGPRs 102 and 103 directly.
[PATCH 3/3] AMDGPU: Fix name of SI control flow lowering source
_
From: Michel Dänzer
Two SGPRs are used for VCC, so it's not possible to use these and VCC
together.
Signed-off-by: Michel Dänzer
---
lib/Target/AMDGPU/SIRegisterInfo.td |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/Target/AMDGPU/SIRegisterInfo.td
b/lib/Target/
Anuj Phogat writes:
> This patch enables support for ETC2 compressed textures on
> all intel hardware. At present, ETC2 texture decoding is not
> available on intel hardware. So, compressed ETC2 texture data
> is decoded in software and stored in a suitable uncompressed
> MESA_FORMAT at the time
Here I thought you were going to remove some dead-code-optimization
passes.
This series is:
Reviewed-by: Eric Anholt
pgpINeK37p6Ty.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/ma
Kenneth Graunke writes:
> Nobody uses it.
For bonus points, this could used the extern opcode_descs[] now that
this was originally copied from.
Patch 8/5?
pgpXg554gOLpx.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesk
56 matches
Mail list logo
