---
src/gallium/state_trackers/clover/api/kernel.cpp | 92 +-
src/gallium/state_trackers/clover/core/module.cpp | 14
src/gallium/state_trackers/clover/core/module.hpp | 11 +++
.../state_trackers/clover/llvm/codegen/common.cpp | 11 +++
4 files changed, 125 insertion
---
src/gallium/state_trackers/clover/llvm/codegen/common.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/state_trackers/clover/llvm/codegen/common.cpp
b/src/gallium/state_trackers/clover/llvm/codegen/common.cpp
index 13ccd59..aa6ca50 100644
--- a/src/gallium/state_tracke
On Sunday 30 October 2016 16:07:25 Francisco Jerez wrote:
> Serge Martin writes:
> > ---
> >
> > src/gallium/state_trackers/clover/api/kernel.cpp | 47
> > --
> > src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++
> >
On Sunday 06 November 2016 17:02:26 Dieter Nützel wrote:
> After latest clover commit 'luxmark-v3.0' sigfault immediately:
Hello
Did you bisect it? Luxmark seems to crash just the same here without this
commit.
Serge
>
> SOURCE/luxmark-v3.0> ./luxmark
> ./luxmark.bin: /usr/local/lib/libOpenCL
On Thursday 06 October 2016 16:26:21 Vedran Miletić wrote:
> CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
> supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for OpenGL.
> CLOVER_CL_C_VERSION_OVERRIDE allows overridng default OpenCL C version.
Thanks.
I've made some
On Thursday 06 October 2016 13:20:36 Jan Vesely wrote:
> On Fri, 2014-12-19 at 16:42 +0100, EdB wrote:
> > CL_PROGRAM_BINARY_TYPE have been added to clGetProgramBuildInfo in
> > CL1.2
> > ---
> > src/gallium/state_trackers/clover/api/program.cpp | 4 +++
> > src/gallium/state_trackers/clover/cor
On Saturday 01 October 2016 15:54:49 Serge Martin wrote:
CC curro
> ---
> src/gallium/state_trackers/clover/api/kernel.cpp | 47 --
> src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++
> src/gallium/state_trackers/clover/core/kernel.hpp | 1 +
&g
On Saturday 01 October 2016 18:51:09 Serge Martin wrote:
> Updated serie, please review.
CC curro
>
> Serge Martin (2):
> clover: add CL_PROGRAM_BINARY_TYPE support (CL1.2)
> clover: add missing clGetDeviceInfo CL1.2 queries
>
> src/gallium/state_trackers/clover/
On Saturday 01 October 2016 19:03:11 Serge Martin wrote:
> On Sunday 27 September 2015 11:15:14 Serge Martin wrote:
> > add clGetExtensionFunctionAddressForPlatform (CL 1.2)
>
> ping (one year reminder :p )
CC curro
>
> > ---
> >
> > src/gallium/state_t
On Monday 03 October 2016 19:37:53 Serge Martin wrote:
> Ping
CC curro
>
> On Saturday 12 September 2015 21:08:20 Serge Martin wrote:
> > Now that mem object can be move back to the host, I think we should latter
> > come with a way to optimize read mapping for such o
Ping
On Saturday 12 September 2015 21:08:20 Serge Martin wrote:
> Now that mem object can be move back to the host, I think we should latter
> come with a way to optimize read mapping for such objets. For the moment if
> they are mapped for reading after been moved to the host, they wil
On Sunday 27 September 2015 11:15:14 Serge Martin wrote:
> add clGetExtensionFunctionAddressForPlatform (CL 1.2)
ping (one year reminder :p )
> ---
> src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +-
> src/gallium/state_trackers/clover/api/dispatch.hpp | 4
&g
CL_PROGRAM_BINARY_TYPE have been added to clGetProgramBuildInfo in CL1.2
---
src/gallium/state_trackers/clover/api/program.cpp | 5 +
src/gallium/state_trackers/clover/core/module.cpp | 1 +
src/gallium/state_trackers/clover/core/module.hpp | 5 +
src/gallium/
---
src/gallium/state_trackers/clover/api/device.cpp | 23 +++
src/gallium/state_trackers/clover/core/device.cpp | 10 ++
src/gallium/state_trackers/clover/core/device.hpp | 2 ++
3 files changed, 35 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/de
Updated serie, please review.
Serge Martin (2):
clover: add CL_PROGRAM_BINARY_TYPE support (CL1.2)
clover: add missing clGetDeviceInfo CL1.2 queries
src/gallium/state_trackers/clover/api/device.cpp | 23 ++
src/gallium/state_trackers/clover/api/program.cpp | 5
---
src/gallium/state_trackers/clover/api/kernel.cpp | 47 --
src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++
src/gallium/state_trackers/clover/core/kernel.hpp | 1 +
src/gallium/state_trackers/clover/core/module.hpp | 19 +--
.../state_trackers/clover/llvm/co
On Wednesday 07 September 2016 19:28:36 Serge Martin wrote:
> On Wednesday 07 September 2016 18:37:51 Vedran Miletić wrote:
> > On 09/07/2016 04:14 PM, Jan Vesely wrote:
> > > On Wed, 2016-09-07 at 13:43 +0200, Vedran Miletić wrote:
> > >> Options specified via the
On Wednesday 07 September 2016 19:17:39 Vedran Miletić wrote:
> The options specified in the CLOVER_EXTRA_COMPILER_OPTIONS shell
> variable are appended to the compiler options specified by the OpenCL
> program, if any.
> Analogously, the options specified in the CLOVER_EXTRA_LINKER_OPTIONS
> varia
On Wednesday 07 September 2016 18:37:51 Vedran Miletić wrote:
> On 09/07/2016 04:14 PM, Jan Vesely wrote:
> > On Wed, 2016-09-07 at 13:43 +0200, Vedran Miletić wrote:
> >> Options specified via the CLOVER_EXTRA_COMPILER_OPTIONS shell
> >> variable
> >> are appended to the compiler and linker option
On Wednesday 31 August 2016 12:39:23 Vedran Miletić wrote:
> On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote:
> > This is required by the OpenCL standard.
> >
> > Signed-off-by: Niels Ole Salscheider
> Reviewed-by: Vedran Miletić
>
> Good catch. Do we miss more defines from [1]?
I think __
R_OPTIONS
> * use debug_get_option
> * append to linker options as well
>
> Signed-off-by: Vedran Miletić
> Reviewed-by: Edward O'Callaghan
> Reviewed-by: Serge Martin
I didn't R-b.
> ---
> docs/envvars.html | 2 ++
> src
This fix getting the size of a struct arg. vec3 types still work ok.
Only buit-in args need to have power of two alignment, getTypeAllocSize
reports the correct size in all cases.
---
This v3 is v1.1 modified for the curent ToT
src/gallium/state_trackers/clover/llvm/codegen/common.cpp | 7 +++---
On Tuesday 30 August 2016 01:20:55 Vedran Miletić wrote:
> Options specified via the CLOVER_COMPILER_OPTIONS shell variable are
> appended to the compiler options specified by the OpenCL program (if
> any).
>
> Signed-off-by: Vedran Miletić
> ---
> docs/envvars.html
On Monday 29 August 2016 22:30:58 Eric Engestrom wrote:
> A few weeks ago, Jose Fonseca suggested [0] we use .editorconfig files
> to try and enforce the formatting of the code, to which Michel Dänzer
> suggested [1] we start by importing the existing .dir-locals.el
> settings. The first draft was
On Thursday 09 June 2016 00:47:27 Serge Martin wrote:
> This fix getting the size of a struct arg. vec3 types still work ok.
> Only buit-in args need to have power of two alignment, getTypeAllocSize
> reports the correct size.
M. I forgot to put V2 and to fix the commit
This fix getting the size of a struct arg. vec3 types still work ok.
Only buit-in args need to have power of two alignment, getTypeAllocSize
reports the correct size.
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src
This fix getting the size of a struct arg. vec3 types still work ok.
Only buit-in args need to have power of two alignment, getTypeAllocSize
reports the correct size.
Cc: 12.0
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
This fix getting the size of a struct arg. vec3 types still work ok.
Only buit-in args need to have power of two alignement, getTypeAllocSize
reports the corect size.
Cc: 12.0
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
patched
with http://reviews.llvm.org/D20168 or an equivalent patch.
Reviewed-by: Serge Martin
Cc: 12.0
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers
On Monday 23 May 2016 09:21:51 Giuseppe Bilotta wrote:
> Code generation (kernel compilation) may sometimes hit LLVM-specific
> bugs. Adding the used LLVM version to the version string may make bug
> triaging easier. (This was inspired by a similar patch recently
> proposed for pocl.)
>
> Signed-o
On Tuesday 17 May 2016 22:54:01 Giuseppe Bilotta wrote:
> On Sun, May 15, 2016 at 10:54 PM, Giuseppe Bilotta
>
> wrote:
> > On Sun, May 15, 2016 at 9:35 AM, Vedran Miletić wrote:
> >> How about also adding Mesa git version, like "OpenGL core profile
> >> version string" has now?
> >
> > That's
me know if this
> >> is not correct -- Or if you think you have contributed a significant
> >> amount of work to this file and I've missed it.
> >
> > The second paragraph probably shouldn't be in the commit log. Either way
> > though,
>
> Sure
On Monday 29 February 2016 12:36:36 Francisco Jerez wrote:
> Serge Martin writes:
> > Ping
>
> Thanks for trying to split some of the LLVM interfacing changes into
> separate patches. I'm about 80% done cleaning up the mess that
> llvm/invocation.cpp is, I'll ta
since 737b6ed13e8f813987b5566004f0f45e9c55f1e8
src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c not longer compile:
error: unknown type name ‘drmDevicePtr’
---
configure.ac | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/configure.ac
index 49be147..55b1c95 10064
Ping
On Saturday 13 February 2016 23:08:34 Serge Martin wrote:
> This serie add clLinkProgram function needed for CL 1.2.
> However, it lacks the binary type part that is mandatory for input
> validation and also for CL_PROGRAM_BINARY_TYPE query. This will be adressed
> in another s
---
src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..8396be9 100644
--- a/src/gallium/state_trackers/clov
Return an API object from an intrusive reference to a Clover object,
incrementing the reference count of the object.
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/util.hpp | 12
1 file changed, 12 insertions(+)
diff --git a/src/gallium/state_trackers/clover
---
.../state_trackers/clover/llvm/invocation.cpp | 87 --
.../state_trackers/clover/llvm/ir_compiler.cpp | 46 +---
.../state_trackers/clover/llvm/ir_compiler.hpp | 4 +-
3 files changed, 85 insertions(+), 52 deletions(-)
diff --git a/src/gallium/state_t
This serie add clLinkProgram function needed for CL 1.2.
However, it lacks the binary type part that is mandatory for input validation
and also for CL_PROGRAM_BINARY_TYPE query. This will be adressed
in another serie once we agree on the way to store it.
Serge Martin (7):
clover: add a LLVM
---
src/gallium/state_trackers/clover/api/program.cpp | 19 --
.../state_trackers/clover/core/compiler.hpp| 7 +-
src/gallium/state_trackers/clover/core/error.hpp | 7 ++
src/gallium/state_trackers/clover/core/program.cpp | 38 ++-
src/gallium/state_trackers/clover/core/pr
/src/gallium/state_trackers/clover/llvm/ir_compiler.cpp
b/src/gallium/state_trackers/clover/llvm/ir_compiler.cpp
new file mode 100644
index 000..8042a3e
--- /dev/null
+++ b/src/gallium/state_trackers/clover/llvm/ir_compiler.cpp
@@ -0,0 +1,282 @@
+// Copyright 2016 Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 234 ++---
1 file changed, 23 insertions(+), 211 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 4d11c24..d3347de 100644
--- a/sr
---
src/gallium/state_trackers/clover/core/program.cpp | 4 ++--
src/gallium/state_trackers/clover/core/program.hpp | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/program.cpp
b/src/gallium/state_trackers/clover/core/program.cpp
index
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/program.cpp | 31 +++---
src/gallium/state_trackers/clover/core/program.hpp | 4 +--
2 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/ga
---
src/gallium/state_trackers/clover/core/kernel.cpp | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..3daec3b 100644
--- a/src/gallium/state_trackers/c
---
src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..a4ef2b1 100644
--- a/src/gallium/state_trackers/clo
On Thursday 04 February 2016 22:25:00 Samuel Pitoiset wrote:
> Signed-off-by: Samuel Pitoiset
Reviewed-by: Serge Martin
> ---
> src/gallium/auxiliary/tgsi/tgsi_transform.h | 2 +-
> src/gallium/auxiliary/tgsi/tgsi_ureg.c | 6 +++---
> 2 files changed, 4 insertions(
Return an API object from an intrusive reference to a Clover object,
incrementing the reference count of the object.
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/util.hpp | 12
1 file changed, 12 insertions(+)
diff --git a/src/gallium/state_trackers/clover
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/program.cpp | 31 +++---
src/gallium/state_trackers/clover/core/program.hpp | 4 +--
2 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/ga
---
src/gallium/state_trackers/clover/api/program.cpp | 19 +-
.../state_trackers/clover/core/compiler.hpp| 7 +-
src/gallium/state_trackers/clover/core/error.hpp | 7 +
src/gallium/state_trackers/clover/core/program.cpp | 41 +++-
src/gallium/state_trackers/clover/core/program.hp
, except for LLVM api change.
Serge Martin (3):
clover: separate compile and link stages
clover: override ret_object
clover: add clLinkProgram (CL 1.2)
src/gallium/state_trackers/clover/api/program.cpp | 50 +++-
src/gallium/state_trackers/clover/api/util.hpp | 12 +
.../state_trackers
src/gallium/tests/trivial/compute.c expects samplers to be cleaned
when the samplers list is NULL.
Like in radeon, the function behave like when the number of samplers
parameter is set to 0.
---
src/gallium/drivers/freedreno/a2xx/fd2_texture.c | 3 +++
src/gallium/drivers/freedreno/a3xx/fd3_textur
in fd_resource_transfer_map, like the others error cases
---
src/gallium/drivers/freedreno/freedreno_resource.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/freedreno/freedreno_resource.c
b/src/gallium/drivers/freedreno/freedreno_resource.c
index 6
in order to avoid returing invalid adress if CALLOC_STRUCT return NULL.
---
src/gallium/drivers/freedreno/freedreno_surface.c | 31 ---
1 file changed, 16 insertions(+), 15 deletions(-)
diff --git a/src/gallium/drivers/freedreno/freedreno_surface.c
b/src/gallium/drivers/freed
On Sunday 29 November 2015 15:23:37 Samuel Pitoiset wrote:
> Nested functions are supported as an extension in GNU C, but Clang
> don't support them.
>
> This fixes compilation errors when (manually) building compute.c,
> or by setting --enable-gallium-tests to the configure script.
>
> Changes f
---
src/gallium/state_trackers/clover/tgsi/compiler.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/state_trackers/clover/tgsi/compiler.cpp
b/src/gallium/state_trackers/clover/tgsi/compiler.cpp
index 54cb747..4d05666 100644
--- a/src/gallium/state_trackers/clover/tgsi/com
changed, this v5 is a rebased v4 in
hope that the first patch in the serie can be reviewed.
CC: Francisco Jerez
Serge Martin (3):
clover: separate compile and link stages
clover: override ret_object
clover: add clLinkProgramm (CL 1.2)
src/gallium/state_trackers/clover/api/program.cpp
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/program.cpp | 31 +++---
src/gallium/state_trackers/clover/core/program.hpp | 4 +--
2 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/ga
Return an API object from an intrusive reference to a Clover object,
incrementing the reference count of the object.
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/util.hpp | 12
1 file changed, 12 insertions(+)
diff --git a/src/gallium/state_trackers/clover
---
src/gallium/state_trackers/clover/api/program.cpp | 19 +-
.../state_trackers/clover/core/compiler.hpp| 7 +-
src/gallium/state_trackers/clover/core/error.hpp | 7 +
src/gallium/state_trackers/clover/core/program.cpp | 41 +++-
src/gallium/state_trackers/clover/core/program.hp
add clGetExtensionFunctionAddressForPlatform (CL 1.2)
---
src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +-
src/gallium/state_trackers/clover/api/dispatch.hpp | 4
src/gallium/state_trackers/clover/api/platform.cpp | 16
3 files changed, 21 insertions(+), 1 deletio
---
src/gallium/state_trackers/clover/api/memory.cpp | 101 ---
1 file changed, 91 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/memory.cpp
b/src/gallium/state_trackers/clover/api/memory.cpp
index 69c3e95..0c2d353 100644
--- a/src/gallium/s
From: Francisco Jerez
This will be used to share the same logic between buffer and image
creation.
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/api/memory.cpp | 80
1 file changed, 41 insertions(+), 39 deletions(-)
diff --git a/src/gallium
Francisco Jerez (1):
clover: Move down canonicalization of memory object flags into
validate_flags().
Serge Martin (2):
clover: implement clCreateImage
clover: implement clCreateImage?D w/ clCreateImage
src/gallium/state_trackers/clover/api/memory.cpp | 215 +--
1
remplace clCreateImage2D and clCreateImage3D implementation with call to
clCreateImage
---
src/gallium/state_trackers/clover/api/memory.cpp | 60
1 file changed, 8 insertions(+), 52 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/memory.cpp
b/src/gallium/
remplace clCreateImage2D and clCreateImage3D implementation with call to
clCreateImage
---
src/gallium/state_trackers/clover/api/memory.cpp | 92 +---
1 file changed, 32 insertions(+), 60 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/memory.cpp
b/src/gallium
---
src/gallium/state_trackers/clover/api/memory.cpp | 117 +--
1 file changed, 107 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/memory.cpp
b/src/gallium/state_trackers/clover/api/memory.cpp
index 1efb95b..6e95931 100644
--- a/src/gallium/
Now that mem object can be move back to the host, I think we should latter
come with a way to optimize read mapping for such objets. For the moment if
they are mapped for reading after been moved to the host, they will be send
back to the device...
Serge Martin (2):
clover
implement memory objects migration to device
---
src/gallium/state_trackers/clover/api/transfer.cpp | 48 ++
src/gallium/state_trackers/clover/core/memory.cpp | 10 ++---
src/gallium/state_trackers/clover/core/memory.hpp | 12 --
.../state_trackers/clover/core/resource.cp
implement memory objects migration back to host
---
src/gallium/state_trackers/clover/api/transfer.cpp | 5 +++-
src/gallium/state_trackers/clover/core/memory.cpp | 28 ++
src/gallium/state_trackers/clover/core/memory.hpp | 4
.../state_trackers/clover/core/resource.cp
On Tuesday 01 September 2015 17:10:33 Albert Freeman wrote:
> Clang tryed to declare the non type member of struct module (enum type type)
> (in clover/core/module.hpp) instead of a variable of type enum (enum type).
>
> Signed-off-by: Albert Freeman
Reviewed by Serge Martin
Can
upport won't be backport to llvm 3.5 this
is good for me. And it no longer make clover crash.
Reviewed-by: Serge Martin (EdB)
But since I don't have commit access, not sure what my r-b worth.
Could it be pushed before the branch point?
> ---
> .../state_trackers/clove
On Sunday 09 August 2015 11:52:13 orome wrote:
> On Sun, 2015-08-09 at 15:39 +0200, Zoltan Gilian wrote:
> > There is no MDOperand in llvm 3.5.
> >
> > v2: Check if kernel metadata is present to avoid crash (EdB).
>
> still builds for me.
Sadly, it's still crashing for me.
auto kernel_node = ge
Hello
This looks like what I've send a few weeks ago [0]. It's seems we should come
with something a little different as explain by curro in [1]
[0] http://lists.freedesktop.org/archives/mesa-dev/2015-August/090479.html
[1] http://lists.freedesktop.org/archives/mesa-dev/2015-June/086110.html
On Friday 07 August 2015 14:45:56 Francisco Jerez wrote:
> "Serge Martin (EdB)" writes:
> > As sugested by Tom a long time ago
> > and in order to be able to create Piglit tests
> >
> > v2:
> > replace NOT_SUPPORTED_BY_CL_1_1 macro with an inline funct
As sugested by Tom a long time ago
and in order to be able to create Piglit tests
v2:
replace NOT_SUPPORTED_BY_CL_1_1 macro with an inline function
remove extra space in clLinkProgram arg
v3:
use __func__
v4:
back to a macro, it make more sense to use it with __func__
---
src/gallium/state_trac
76 matches
Mail list logo