Fixes new piglit test:
tests/spec/glsl-1.20/execution/qualifiers/vs-out-conversion-int-to-float-vec4-index.shader_test
---
src/compiler/glsl/ast_function.cpp | 94 ++
1 file changed, 94 insertions(+)
diff --git a/src/compiler/glsl/ast_function.cpp
b/src/compiler/glsl/
On 12/07/18 19:00, Michel Dänzer wrote:
On 2018-07-12 07:26 AM, Marek Olšák wrote:
From: Marek Olšák
TODO: requires latest libdrm for amdgpu_bo_handle_type_kms_noimport
---
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 6 +++
src/gallium/winsys/amdgpu/drm/amdgpu_bo.h | 2 +
src/gallium/w
Two suggestions below.
On 12/07/18 15:26, Marek Olšák wrote:
From: Marek Olšák
This decreases sizeof(struct amdgpu_cs_buffer) from 24 to 16 bytes.
---
src/gallium/drivers/radeon/radeon_winsys.h| 39 ++-
src/gallium/drivers/radeonsi/si_debug.c | 2 +-
src/gallium/
On 12/07/18 15:26, Marek Olšák wrote:
From: Marek Olšák
TODO: requires latest libdrm for amdgpu_bo_handle_type_kms_noimport
Forgot to bump this?
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo
On 12/07/18 07:18, Jason Ekstrand wrote:
I sent out a series for this almost a year ago and it just sat on the list
rotting away. You can find the original series here:
https://patchwork.freedesktop.org/series/32809/
This v2 is a rebase of that series. I believe Tim reviewed most of the
origi
On 11/07/18 19:45, Eero Tamminen wrote:
Hi,
On 11.07.2018 12:00, Timothy Arceri wrote:
On 11/07/18 18:20, Eero Tamminen wrote:
Have you considered partial loop unrolling support?
I.e. when loop counter is known, but too high for full unroll, doing
partial loop unrolling (e.g. unroll 4x
nroll limits on the Talos
benchmark for RADV. I guess it might be interesting to try partial
unrolling with that Game, it would be good to know where else it might
help.
- Eero
On 11.07.2018 09:48, Timothy Arceri wrote:
This series started out as me trying to unrolls some useless lo
In GLSL IR we cheat with switch statements and simply convert them
into loops with a single iteration. This allowed us to make use of
the existing jump instruction handling provided by the loop handing
code, it also allows dead code to be cleaned up once we have
wrapped the code in a loop.
However
This adds support for unrolling the classic
do {
// ...
} while (false)
that is used to wrap multi-line macros. GLSL IR also wraps switch
statements in a loop like this.
shader-db results IVB:
total loops in shared programs: 2515 -> 2512 (-0.12%)
loops in affected programs: 33 -
Now that SSA values can be derefs and they have special rules, we have
to be a bit more careful about our LCSSA phis. In particular, we need
to clean up in case LCSSA ended up creating a phi node for a deref.
This avoids validation issues with some CTS tests with the new patch,
but its possible th
Here we replace one side of the ior with NIR_TRUE if the src is a
loop terminators condition that we know can only be true.
No shader-db change.
---
src/compiler/nir/nir_opt_if.c | 28 +---
1 file changed, 25 insertions(+), 3 deletions(-)
diff --git a/src/compiler/nir/nir
This allows some loops to unroll were they are guaranteed to
exit after the first iteration. For example:
loop {
block block_1:
/* preds: block_0 block_13 */
vec1 32 ssa_85 = load_const (0x0002 /* 0.00 */)
vec1 32 ssa_
This will help later patches with unrolling loops that end with a
break i.e. loops the always exit on their first interation.
---
src/compiler/nir/nir_loop_analyze.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/compiler/nir/nir_loop_analyze.c
b/src/compil
In order to be sure loop_terminator_list is an accurate
representation of all the jumps in the loop we need to be sure we
didn't encounter any other complex behaviour such as continues,
nested breaks, etc during analysis.
This will be used in the following patch.
---
src/compiler/nir/nir.h
For simple loop terminators we can evaluate all further uses of the
condition in the loop because we know we must have either exited
the loop or we have a known value.
shader-db results IVB (all changes from dolphin uber shaders):
total instructions in shared programs: 10022822 -> 10018187 (-0.05
The innermost check was added to stop us from unrolling multiple
loops in a single pass, and to stop outer loops from unrolling.
When we successfully unroll a loop we need to run the analysis
pass again before deciding if we want to go ahead an unroll a
second loop.
However the logic was flawed b
This series started out as me trying to unrolls some useless loops I
spotted in some shaders from DXVK games (see patch 10), but I found
some other issues and improvements along the way.
The biggest winner seem like it could be the dolphin uber shaders on
i965 (on radeonsi the shaders don't seem t
Since we know what side of the branch we ended up on we can just
replace the use with a constant.
All helped shaders are from Unreal Engine 4 besides one shader from
Dirt Showdown.
shader-db results SKL:
total instructions in shared programs: 13219725 -> 13219643 (<.01%)
instructions in affected
Reviewed-by: Timothy Arceri
On 04/07/18 11:51, srol...@vmware.com wrote:
From: Roland Scheidegger
Empty initializer braces aren't valid c (it's a gnu extension, and
it's valid in c++).
Hopefully fixes appveyor / msvc build...
Fixes 6677e131b806b10754adcb7cf3f427a7fcc2aa09
---
Its unlikely anyone will add proper ARB_direct_state_access compat
support before we branch 18.2. Enabling the extension in 4.5 at
least allows users to make use of MESA_GL_VERSION_OVERRIDE=4.5COMPAT
for games like No Mans Sky.
---
src/mapi/glapi/gen/apiexec.py| 194 +++
The game forgets to enable multiple extensions in its shaders, one
of those extesions is EXT_texture_array. But enabling this config
entry fixes at least one other rendering issue that enabling
EXT_texture_array on its own doesn't fix.
---
src/util/drirc | 4
1 file changed, 4 insertions(+)
Series:
Reviewed-by: Timothy Arceri
On 04/07/18 09:16, Marek Olšák wrote:
From: Marek Olšák
There is a 15-character limit for thread names shared by the queue name
and process name. Shorten the thread name to make space for the process
name.
---
src/gallium/auxiliary/util
On 03/07/18 05:51, Eric Anholt wrote:
Eric Anholt writes:
[ Unknown signature status ]
Timothy Arceri writes:
nir_compact_varyings() is meant to skip over varyings used by xfb:
/* We can't repack xfb varyings. */
if (var->data.always_a
Series looks good to me. How much to go before we can turn this
extension on?
On 03/07/18 00:58, Alejandro Piñeiro wrote:
Hi Timothy. Thanks for the quick review!
As you suggested some squash and commit drops, Im resending the v2 of
the series, just in case you want a final overview of the ser
Thanks.
Reviewed-by: Timothy Arceri
On 03/07/18 00:58, Alejandro Piñeiro wrote:
From: Antia Puentes
From the SPIR-V 1.0 specification, section 3.32.18, "Atomic
Instructions":
"OpAtomicIDecrement:
The instruction's result is the Original Value
you missed my r-b here
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Timothy Arceri
On 02/07/18 05:50, Marek Olšák wrote:
From: Marek Olšák
Users are encouraged to switch to LLVM 6.0 released in March 2018.
---
.travis.yml | 24 ++--
configure.ac | 4 +-
meson.build
Reviewed-by: Timothy Arceri
On 30/06/18 00:29, Alejandro Piñeiro wrote:
From: Antia Puentes
This will perform the CS shared lowering. See 8761a04d0d93
---
src/mesa/main/glspirv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/glspirv.c b/src/mesa/main/glspirv.c
index
Reviewed-by: Timothy Arceri
On 30/06/18 00:29, Alejandro Piñeiro wrote:
From: Neil Roberts
---
src/mesa/drivers/dri/i965/brw_link.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp
b/src/mesa/drivers/dri/i965/brw_link.cpp
index 8bc97fa4f3e
Reviewed-by: Timothy Arceri
On 30/06/18 00:29, Alejandro Piñeiro wrote:
That is the same gen requirement for ARB_shader_atomic_counters.
---
src/mesa/drivers/dri/i965/brw_context.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa
On 30/06/18 00:29, Alejandro Piñeiro wrote:
From: Antia Puentes
From the SPIR-V specification, OpAtomicIDecrement:
"The instruction's result is the Original Value."
However, we were implementing it, for uniform atomic counters, as a
pre-decrement operation.
Renamed the former nir intrinsic
patches 10-14 are:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Patches 4-7:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
There is no need to pass storage_class to this function. We should
just let the caller check that which it does.
If you agree with the changes. This patch is:
Reviewed-by: Timothy Arceri
+
+ return atomic;
+}
+
nir_deref_instr *
vtn_pointer_to_deref(struct vtn_builder *b, struct vtn_p
NAK this shouldn't be needed.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
UniformRemapTable += entries;
+ unsigned chosen_location =
Please move patch 2 to patch 1 and squash with this one with the
current patch 1. This is just code churn.
As with my review of patch 1 please rename chosen_location -> location
Otherwise:
Reviewed-by: Timothy Arceri
+
+++ b/src/compiler/glsl/linker_util.cpp
@@ -24,6 +24,7 @@
#include "main/mtypes.h"
#include "linker_util.h"
#include "util/set.h"
+#include "compiler/glsl/ir_uniform.h" /* for gl_uniform_storage */
This only needs to be #include "ir_uniform.h"
With that this patch is:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 30/06/18 16:05, Timothy Arceri wrote:
On 30/06/18 00:28, Alejandro Piñeiro wrote:
ARB_gl_spirv points that uniforms in general need explicit
location. But there are still some cases of uniforms without location,
like for example uniform atomic counters. Those doesn't have a
location
and use those first.
*/
+ unsigned chosen_location = prog->NumUniformRemapTable;
Can we please just rename chosen_location
With those two nits this patch is:
Reviewed-by: Timothy Arceri
+
+ /* resize remap table to fit new entries */
+ prog->UniformRem
Series:
Reviewed-by: Timothy Arceri
On 30/06/18 13:44, Jason Ekstrand wrote:
---
src/intel/vulkan/anv_device.c | 36 ++
src/intel/vulkan/anv_pipeline_cache.c | 98 ---
src/intel/vulkan/anv_private.h| 3 +
3 files changed, 126 insertions
Reviewed-by: Timothy Arceri
On 30/06/18 14:59, Marek Olšák wrote:
From: Marek Olšák
Shaders that need special code for external samplers were broken if
they were loaded from the cache.
Cc: 18.1
---
src/compiler/glsl/serialize.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src
src_deref = nir_src_as_deref(intrin->src[0]);
+break;
+
+ case nir_intrinsic_copy_deref:
Can we add a comment here? Something like:
/* We always assume the src and therefore the dst are not constants
* here. Copy and constant propagation passes should have taken
I've sent some minor nits. Otherwise the series looks good, although I
did only skim most of the actual new i965 blob changes. Series:
Reviewed-by: Timothy Arceri
You might want to wait for Tapani to take a look also.
On 15/05/18 02:52, Jordan Justen wrote:
Signed-off-by: Jordan J
On 15/05/18 02:52, Jordan Justen wrote:
We will need to populate the default key for ARB_get_program_binary to
allow us to retrieve the default gen program to store save in the
store save - > store ???
program binary.
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_cs.c
On 15/05/18 02:52, Jordan Justen wrote:
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_program_cache.c | 15 +++
src/mesa/drivers/dri/i965/brw_state.h | 2 ++
2 files changed, 17 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_program_cache.c
b
The subject line contains i965:
On 15/05/18 02:52, Jordan Justen wrote:
Signed-off-by: Jordan Justen
---
src/mesa/state_tracker/st_context.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/state_tracker/st_context.c
b/src/mesa/state_tracker/st_context.c
index ee76e07a7d1..0d0
s exposing functions
incorrectly). Guess its back to adding another drirc entry and config
option.
On Fri, Jun 29, 2018 at 12:42 AM, Timothy Arceri wrote:
This extension was made core in OpenGL 3.0.
This fixes rendering issues in No Man's Sky.
---
src/compiler/glsl/builtin_functions.cpp |
v2: add missing ARB_base_instance support
---
src/mesa/main/extensions_table.h | 2 +-
src/mesa/vbo/vbo_exec_array.c| 77 +++-
2 files changed, 76 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index
v2: add missing ARB_base_instance support
---
src/mesa/main/bufferobj.c| 3 +-
src/mesa/main/extensions_table.h | 2 +-
src/mesa/vbo/vbo_exec_array.c| 71 +++-
3 files changed, 72 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/bufferobj.c b/src/
This extension was made core in OpenGL 3.0.
This fixes rendering issues in No Man's Sky.
---
src/compiler/glsl/builtin_functions.cpp | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/compiler/glsl/builtin_functions.cpp
b/src/compiler/glsl/builtin_functions.cpp
index
The extension is enabled for compat profile but there is currently
no display list support.
I filed a spec bug and it has been agreed that
glDispatchComputeIndirect should generate an INVALID_OPERATION
error when called during display list compilation.
---
src/mesa/main/dlist.c | 39 +
---
src/gallium/drivers/radeonsi/si_get.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_get.c
b/src/gallium/drivers/radeonsi/si_get.c
index 0e8617d0fee..96ff2a9e46b 100644
--- a/src/gallium/drivers/radeonsi/si_get.c
+++ b/src/gallium/dr
---
src/mapi/glapi/gen/apiexec.py | 20 ++--
src/mesa/main/extensions_table.h| 2 +-
src/mesa/main/tests/dispatch_sanity.cpp | 23 ---
src/mesa/main/vtxfmt.c | 2 +-
4 files changed, 24 insertions(+), 23 deletions(-)
diff --
From: Dave Airlie
Required for ARB_vertex_attrib_64bit compat profile support.
---
src/mesa/main/mtypes.h | 2 +-
src/mesa/vbo/vbo_private.h | 3 +++
src/mesa/vbo/vbo_save_api.c | 18 --
src/mesa/vbo/vbo_save_draw.c | 18 +-
4 files changed, 29 insertio
---
src/mesa/main/bufferobj.c| 3 +-
src/mesa/main/extensions_table.h | 2 +-
src/mesa/vbo/vbo_exec_array.c| 66 +++-
3 files changed, 67 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 67f9cd0a902..1
---
src/mesa/main/dlist.c | 178 ++
1 file changed, 178 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 5ff0a23018c..ae23d292837 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -471,6 +471,10 @@ typedef enum
---
src/mesa/main/dlist.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index d49eebae00d..2425cf24f1b 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -523,6 +523,9 @@ typedef enum
/* ARB_
The spec doesn't explicitly say to generate an error but since
DrawArraysInstanced* and DrawElementsInstanced* do, it makes
sense to do it for these functions also.
---
src/mesa/main/dlist.c | 47 +++
1 file changed, 47 insertions(+)
diff --git a/src/mesa/m
---
src/mesa/main/extensions_table.h | 2 +-
src/mesa/vbo/vbo_exec_array.c| 75 +++-
2 files changed, 74 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index 1446a4bd421..12b796777df 100644
--- a/src/
---
src/mapi/glapi/gen/apiexec.py | 16
src/mesa/main/extensions_table.h| 2 +-
src/mesa/main/tests/dispatch_sanity.cpp | 17 +
3 files changed, 18 insertions(+), 17 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/gen/ap
We will use this to add compat support to ARB_multi_draw_indirect
in the following patch.
---
src/mesa/main/draw_validate.c | 24
src/mesa/main/draw_validate.h | 3 +++
2 files changed, 15 insertions(+), 12 deletions(-)
diff --git a/src/mesa/main/draw_validate.c b/src/me
---
src/mapi/glapi/gen/apiexec.py | 16
src/mesa/main/extensions_table.h| 2 +-
src/mesa/main/tests/dispatch_sanity.cpp | 19 +--
3 files changed, 18 insertions(+), 19 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/gen/
This is required for fp64 to be enabled in compat profile.
---
src/mesa/main/dlist.c | 514 ++
1 file changed, 514 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index aec373b7ab1..d49eebae00d 100644
--- a/src/mesa/main/dlist.c
+++
---
src/mesa/main/extensions_table.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index f04fea9e3bc..f79a52cee8c 100644
--- a/src/mesa/main/extensions_table.h
+++ b/src/mesa/main/extensions_table.h
@@ -1
---
src/mesa/main/dlist.c | 211 ++
1 file changed, 211 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 2425cf24f1b..b2b1f723a17 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -290,6 +290,15 @@ typedef enum
Sorry to keep spamming the list with this stuff, but Dave helped
out with ARB_vertex_attrib_64bit support and the spec bug I
submitted for indirect compute dispatch was resolved so it
seemed like a good idea to send it all out again together with
these updates.
Pretty much everything has correspon
This is required so we can enable fp64 support in compat profile.
---
src/mapi/glapi/gen/apiexec.py | 36 +--
src/mesa/main/dlist.c | 493 ++
2 files changed, 511 insertions(+), 18 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/ge
---
src/mesa/main/extensions_table.h| 2 +-
src/mesa/main/tests/dispatch_sanity.cpp | 38 -
2 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index 7af48a4ad91..5fe2e88fe98 100644
--
I managed to miss these two in my last pass at this.
---
src/mesa/main/get_hash_params.py | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_params.py
index 83136108e95..618e306e509 100644
--- a/src/mesa/main/get_hash_
The extension is enabled for compat profile but there is currently
no display list support.
I filed a spec bug and it has been agreed that
glDispatchComputeIndirect should generate an INVALID_OPERATION
error when called during display list compilation.
---
v2: Generate an error for glDispatchCom
On 28/06/18 09:23, Timothy Arceri wrote:
The spec allows adding scalars a vector or matrix. In this case
the opt was losing any swizzle or size information.
This fixes a bug with Doom (2016) shaders.
Forgot to add:
Fixes: 34ec1a24d61f ("glsl: Optimize (x + y cmp 0) into (x c
The spec allows adding scalars a vector or matrix. In this case
the opt was losing any swizzle or size information.
This fixes a bug with Doom (2016) shaders.
---
src/compiler/glsl/opt_algebraic.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/compiler/glsl/opt_algebraic.cpp
b/s
On 26/06/18 09:51, Dieter Nützel wrote:
Hello Marek,
after this series landed I get this:
Making all in targets/pipe-loader
make[4]: Verzeichnis „/opt/mesa/src/gallium/targets/pipe-loader“ wird
betreten
CXXLD pipe_r600.la
../../../../src/gallium/winsys/radeon/drm/.libs/libradeonwinsys.a
Please note I've fixed up make check where is was broken when enabling
some of these extensions. You can see all of the latests compat patches
here:
https://gitlab.freedesktop.org/tarceri/mesa/commits/ARB_gpu_shader_fp64
On 22/06/18 18:47, Timothy Arceri wrote:
I've send all of the
The spec doesn't explicitly say to generate an error but since
DrawArraysInstanced* and DrawElementsInstanced* do, it makes
sense to do it for these functions also.
---
src/mesa/main/dlist.c | 47 +++
1 file changed, 47 insertions(+)
diff --git a/src/mesa/m
---
src/mesa/main/bufferobj.c| 3 +-
src/mesa/main/extensions_table.h | 2 +-
src/mesa/vbo/vbo_exec_array.c| 66 +++-
3 files changed, 67 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 67f9cd0a902..1
---
src/gallium/drivers/radeonsi/si_get.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_get.c
b/src/gallium/drivers/radeonsi/si_get.c
index 0e8617d0fee..a99626416e7 100644
--- a/src/gallium/drivers/radeonsi/si_get.c
+++ b/src/gallium/drivers/r
We will use this to add compat support to ARB_multi_draw_indirect
in the following patch.
---
src/mesa/main/draw_validate.c | 24
src/mesa/main/draw_validate.h | 3 +++
2 files changed, 15 insertions(+), 12 deletions(-)
diff --git a/src/mesa/main/draw_validate.c b/src/me
---
src/mesa/main/extensions_table.h | 2 +-
src/mesa/vbo/vbo_exec_array.c| 75 +++-
2 files changed, 74 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index cba1ef6ba7c..1a4014c646b 100644
--- a/src/
This series is intended to be applied on top of my previous compat
series [1]. Note I'm going to drop the last two patches from that
series, patch 10 was sent by mistake and I've submitted a spec bug
in regards to patch 11.
I originally planned not to send this series until
ARB_vertex_attrib_6
Whoops please ignore this patch it was sent by mistake.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
on 4.4
On Fri, 22 Jun 2018 at 09:48 Timothy Arceri <mailto:tarc...@itsqueeze.com>> wrote:
I've send all of these patches out already. The only change is to the
first 2 patches which fixes things so we store a double as 2
unsigned ints
rather than 2 floats.
E
On 22/06/18 18:47, Timothy Arceri wrote:
The extension is enabled for compat profile but there is currently
no display list support.
---
src/mesa/main/dlist.c | 87 +++
1 file changed, 87 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa
Reviewed-by: Timothy Arceri
On 23/06/18 06:11, Eric Anholt wrote:
The always_active_io flag was only set according to the first variable
that got packed in, so NIR io compaction would end up compacting XFB
varyings that shouldn't move at that point.
---
This doesn't fix my XFB iss
In the subject line you need to change:
ac_build_gather_values -> lp_build_gather_values
Series:
Reviewed-by: Timothy Arceri
On 23/06/18 08:31, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 45 +--
.../drivers/radeo
This does more than moving. Can you add a commit message about why its
safe to change cmask_size from uint64_t -> uint32_t
On 23/06/18 08:32, Marek Olšák wrote:
From: Marek Olšák
---
src/amd/common/ac_surface.c | 6 +++---
src/amd/common/ac_surface.h | 16 +
---
src/mesa/main/extensions_table.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index bc1f21a2926..b1ee0214d6c 100644
--- a/src/mesa/main/extensions_table.h
+++ b/src/mesa/main/extensions_table.h
@@ -1
The extension is enabled for compat profile but there is currently
no display list support.
---
src/mesa/main/dlist.c | 87 +++
1 file changed, 87 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index b2b1f723a17..c11b4c06fe5 100644
---
src/mapi/glapi/gen/apiexec.py| 16
src/mesa/main/extensions_table.h | 2 +-
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/gen/apiexec.py
index e69c6b4df16..1a91785d375 100644
--- a/src/mapi/glapi/gen/apiexec.
---
src/mesa/main/dlist.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index d49eebae00d..2425cf24f1b 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -523,6 +523,9 @@ typedef enum
/* ARB_
---
src/mesa/main/extensions_table.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index 1c55df8a228..423f22b4657 100644
--- a/src/mesa/main/extensions_table.h
+++ b/src/mesa/main/extensions_table.h
@@ -76,7
---
src/mapi/glapi/gen/apiexec.py| 20 ++--
src/mesa/main/extensions_table.h | 2 +-
2 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/gen/apiexec.py
index 1a91785d375..44552f43f29 100644
--- a/src/mapi/glapi/gen/ap
I've send all of these patches out already. The only change is to the
first 2 patches which fixes things so we store a double as 2 unsigned ints
rather than 2 floats.
Everything here has corresponding piglit tests all of which are now
upstream with the exception of the compute shader display list
---
src/mapi/glapi/gen/apiexec.py| 16
src/mesa/main/extensions_table.h | 2 +-
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/gen/apiexec.py
index 00c80171274..e69c6b4df16 100644
--- a/src/mapi/glapi/gen/apiexec.
---
src/mesa/main/dlist.c | 211 ++
1 file changed, 211 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 2425cf24f1b..b2b1f723a17 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -290,6 +290,15 @@ typedef enum
This is required so we can enable fp64 support in compat profile.
---
src/mapi/glapi/gen/apiexec.py | 36 +--
src/mesa/main/dlist.c | 493 ++
2 files changed, 511 insertions(+), 18 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/ge
I managed to miss these two in my last pass at this.
---
src/mesa/main/get_hash_params.py | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_params.py
index 83136108e95..618e306e509 100644
--- a/src/mesa/main/get_hash_
This is required for fp64 to be enabled in compat profile.
---
src/mesa/main/dlist.c | 514 ++
1 file changed, 514 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index aec373b7ab1..d49eebae00d 100644
--- a/src/mesa/main/dlist.c
+++
The extension is enabled for compat profile but there is currently
no display list support.
---
src/mesa/main/dlist.c | 87 +++
1 file changed, 87 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 8b1ddb05038..e5c8f22ac80 100644
Reviewed-by: Timothy Arceri
On 21/06/18 11:02, Marek Olšák wrote:
From: Marek Olšák
Cc: 18.1
---
src/mesa/main/get.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 772ca00da1f..db0079beb51 100644
--- a/src/mesa/main
701 - 800 of 6697 matches
Mail list logo