Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/core/kernel.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 4716705323..328323b6b0
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/program.cpp | 35 ---
1 file changed, 25 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state_trackers/clov
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/core/device.cpp | 11 +++
src/gallium/state_trackers/clover/core/device.hpp | 3 +++
2 files changed, 14 insertions(+)
diff --git a/src/gallium/state_trackers/clover/core/device.cpp
b/src/g
OpenCL 2.1 gives the ability to query for a program’s IL, which is
returned as a pointer.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
.../state_trackers/clover/core/property.hpp| 39 ++
1 file changed, 39 insertions(+)
diff --git a/src/g
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/core/program.cpp | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/program.cpp
b/src/gallium/state_trackers/clove
Creating a program using clCreateProgramWithSource to SPIR-V requires a
non-upstreamed version of LLVM and clang, therefore it is currently not
supported.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/core/program.cpp | 4
1 file chan
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/program.cpp | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state_trackers/clover/api/program.cpp
index 57b8
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/program.cpp | 29 ++-
src/gallium/state_trackers/clover/core/program.cpp | 57 --
src/gallium/state_trackers/clover/core/program.hpp | 14 ++
3 files chang
If a field was not present, its value will default to 0.
This lets the code run on outputs that did not include the amount of
shared memory used, for example.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
nv-report.py | 38 --
1 file chang
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
nv-report.py | 22 ++
1 file changed, 14 insertions(+), 8 deletions(-)
diff --git a/nv-report.py b/nv-report.py
index bb6b42d..7bd3356 100644
--- a/nv-report.py
+++ b/nv-report.py
@@ -3,8 +3,8 @@
""&
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
nv-report.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/nv-report.py b/nv-report.py
index abd6110..724c662 100644
--- a/nv-report.py
+++ b/nv-report.py
@@ -62,7 +62,12 @@ def analyze(fname):
return
In OpenCL/CUDA kernels, shared memory usage can be defined within the
kernel code. Those usage will only be picked up while parsing the
SPIR-V, during the translation phase of the program.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/drivers/nouveau/nv50/nv50_pro
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/drivers/nouveau/nv50/nv50_program.c | 7 ---
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 7 ---
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nv50/nv50_program.c
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h
b/src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h
index 76f0
he end. Either way.
I placed it there to group it with other memory storage information such as
local, and gpr in some way.
I’ll send a patch to update nv-report (to which ML should I send it?).
> On Mon, Oct 2, 2017 at 2:57 PM, Pierre Moreau <pierre.mor...@free.fr> wrote:
>
int shl 36
uniform int shr 36
uniform int64_t iexpected_shl 0x09ac9010
uniform int64_t iexpected_shr -0x7dfcff0 # bit pattern: 0xf8203010
uniform uint64_t uexpected_shl 0x50102030
uniform uint64_t uexpected_shr 0x0140
draw rect ortho 12 0 4 4
Signed-off-by: Pie
The existing lowering code assumed the shift would not be an immediate
but did not guard against it. However, in the constant folding pass, a
multiplication by a power-of-2 immediate would get optimised into a
shift-left instruction, with the shift value being an immediate.
Signed-off-by: Pierre
On 2017-12-03 — 15:36, Ilia Mirkin wrote:
> On Sun, Dec 3, 2017 at 3:28 PM, Pierre Moreau <pierre.mor...@free.fr> wrote:
> > The existing lowering code assumed the shift would not be an immediate
> > but did not guard against it. However, in the constant folding pass,
Those operations do not map to actual hardware instructions, therefore
those should always be lowered to 32-bit instructions.
Fixes: 009c54aa7af "nv50/ir: Split 64-bit integer MAD/MUL operations"
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/drivers
Thank you for the changes!
Reviewed-by: Pierre Moreau <pierre.mor...@free.fr>
On 2017-12-07 — 17:12, Alejandro Piñeiro wrote:
> To avoid any vulkan driver to include the GL mtypes.h. Renamed as
> eventually this could be used by drivers not using nir.
>
> v2: remove compiler/
Hello Alejandro,
As far as I understand, nir_spirv_supported_capabilities is being filled in by
the driver and then fetched by the API entrypoint to check the capabilities
required by the SPIR-V binary given as input. And this is done regardless of
the input IR used by the driver, be it NIR, LLVM
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Changes in:
- v7: Update the dependency name (meson) and the libs variable (Makefile) due
to the replacement of llvm-spirv to the new official SPIRV-LLVM-Translator
src/gallium/state_trackers/clover/Makefile.am
clover: Implement clCreateProgramWithILKHR;
* and the three patches I just resent as v7.
Thank you.
Pierre
On 2018-03-25 — 20:02, Pierre Moreau wrote:
> Hello,
>
> This has only minor changes compared to the previous version, but as it
> introduces a new patch, I preferred to resen
On 2018-05-23 — 12:12, Eric Engestrom wrote:
> On Tuesday, 2018-05-22 23:06:16 +0200, Karol Herbst wrote:
> > From: Pierre Moreau <pierre.mor...@free.fr>
> >
> > Signed-off-by: Karol Herbst <kher...@redhat.com>
> > ---
> &
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Changes in:
- v7: Update the LLVM API call due to the replacement of llvm-spirv to the new
official SPIRV-LLVM-Translator
src/gallium/state_trackers/clover/Makefile.am | 6 +++-
.../state_trackers/clover/llvm/invocation.cp
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Changes in:
- v7: Replace the llvm-spirv repository by the new official
SPIRV-LLVM-Translator
configure.ac | 18 ++
meson.build | 8
2 files changed, 26 insertions(+)
diff --git a/configur
On 2018-05-23 — 15:51, Dylan Baker wrote:
> Quoting Pierre Moreau (2018-05-23 14:43:10)
> > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
> > ---
> > Changes in:
> > - v7: Replace the llvm-spirv repository by the new official
> > SPIRV-LLVM
Signed-off-by: Pierre Moreau
---
Notes:
Changes in:
* v8: Add DEFINES to libclspirv and libclover, in autotools, as they would
otherwise never know whether CLOVER_ALLOW_SPIRV has been defined
(Dave Airlie)
* v7: Update the dependency name (meson) and the libs variable
Reviewed-by: Karol Herbst
Reviewed-by: Dylan Baker
Signed-off-by: Pierre Moreau
---
Notes:
Changes in:
* v8: * Properly align LLVMSPIRVLib comment (Dylan Baker)
* Only define CLOVER_ALLOW_SPIRV when **both** dependencies are found:
autotools was only requiring one
The cap would need to be added to the documentation as well, in
“src/gallium/docs/source/screen.rst”.
I might be wrong, but I think you are going to break all existing drivers in
clover, that do not yet support the new cap: for unsupported caps, drivers
return a value of 0, which means they would
Series is
Reviewed-by: Pierre Moreau <pierre.mor...@free.fr>
On 2017-12-16 — 23:00, Rhys Kidd wrote:
> Couple of little compiler warning cleanups so that nouveau builds without any
> warnings for meson's debug and release builds with gcc 7.2.0
>
> v2: Add preventative co
With the comments below addressed, this patch is
Reviewed-by: Pierre Moreau <pierre.mor...@free.fr>
On 2018-01-04 — 16:01, Karol Herbst wrote:
> v2: remove TGSI related bits
>
> Signed-off-by: Karol Herbst <kher...@redhat.com>
> ---
> src/gallium/drivers/nouveau/M
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/program.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state_trackers/clover/api/program.cpp
index ed3b679c7c..754a
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 2 +-
src/gallium/state_trackers/clover/api/program.cpp | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +-
src/gallium/state_trackers/clover/api/program.cpp | 8
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/api/dispatch.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
b/src/gallium/state_trackers/clover/api/device.cpp
index 4e274c5005..6bed
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/dispatch.hpp | 4 ++
src/gallium/state_trackers/clover/api/program.cpp | 29 -
src/gallium/state_trackers/clover/core/program.cpp | 68 +-
src/gallium/state_trackers/
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/platform.cpp | 2 ++
src/gallium/state_trackers/clover/core/device.cpp | 3 ++-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/api/platform.cpp
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
include/CL/cl.h | 8
include/CL/cl_platform.h | 1 +
2 files changed, 9 insertions(+)
diff --git a/include/CL/cl.h b/include/CL/cl.h
index 316565d6e4..2130c19c5e 100644
--- a/include/CL/cl.h
+++ b/include/C
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Resending to the ML, as gabe ran out of memory.
src/gallium/state_trackers/clover/Makefile.am | 15 +-
src/gallium/state_trackers/clover/Makefile.sources | 4 +
src/gallium/state_trackers/clover/api/program.cpp | 1
Creating a program using clCreateProgramWithSource to SPIR-V requires a
non-upstreamed version of LLVM and clang, therefore it is currently not
supported.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Resending to the ML, as gabe ran out of memory.
src/gallium/state_trackers/
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Resending to the ML, as gabe ran out of memory.
src/gallium/state_trackers/clover/core/program.cpp | 22 +-
1 file changed, 17 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/clove
On 2018-01-23 — 15:09, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> > On 2018-01-23 — 14:03, Francisco Jerez wrote:
> >> Pierre Moreau <pierre.mor...@free.fr> writes:
> >>
> >> > From the OpenCL
On 2018-01-24 — 09:19, Tomeu Vizoso wrote:
> On 01/24/2018 12:03 AM, Karol Herbst wrote:
> > On Tue, Jan 23, 2018 at 11:46 PM, Francisco Jerez <curroje...@riseup.net>
> > wrote:
> > > Pierre Moreau <pierre.mor...@free.fr> writes:
> > >
> &g
On 2018-01-23 — 14:12, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
> > ---
> > src/gallium/state_trackers/clover/api/device.cpp | 5 +
> > 1 file changed, 5 inse
On 2018-02-02 — 18:07, Timothy Arceri wrote:
>
>
> On 02/02/18 17:21, Timothy Arceri wrote:
> > On 02/02/18 16:38, Jan Vesely wrote:
> > > On Fri, 2018-02-02 at 15:03 +1100, Timothy Arceri wrote:
> > > > When PIPE_SHADER_IR_LLVM existed this query made sense but now it
> > > > always returns
(Moving the conversation to its own thread.)
> target agnostic libclc is rather difficult to do. CLC includes 3 levels
> of precision on float (fp32) operands; full, half, native. The
> implementation of each depends on capabilites of specific device (e.g.
> vega(VI+?) can do 1 ULP log2/exp2 in
On 2018-02-08 — 12:47, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> > On 2018-02-07 — 12:36, Francisco Jerez wrote:
> >> Pierre Moreau <pierre.mor...@free.fr> writes:
> >>
> >> > On 2018-02-06 — 20:50, Jan
Reviewed-by: Pierre Moreau <pierre.mor...@free.fr>
On 2018-02-08 — 10:57, Timothy Arceri wrote:
> PIPE_SHADER_CAP_PREFERRED_IR was conlicting with PIPE_SHADER_IR_NIR
> for compute shaders, so we let clover pick the one it wants to use.
>
> This change indirectly enables NIR s
On 2018-02-07 — 12:36, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> > On 2018-02-06 — 20:50, Jan Vesely wrote:
> > [snip]
> >> > > Happy to here suggestions for solving the current conflict in uses of
> >> > > P
IR_TGSI;
> }
> - case PIPE_SHADER_CAP_SUPPORTED_IRS:
> + return PIPE_SHADER_IR_TGSI;
This line is dead code and can be safely removed, as both the if-block and the
else-block return a value already; it should be part of patch 03 instead. With
that addressed, this patch is
On 2018-02-09 — 12:06, Timothy Arceri wrote:
> PIPE_SHADER_CAP_PREFERRED_IR was conflicting with PIPE_SHADER_IR_NIR
> for compute shaders, so we let clover pick the one it wants to use.
> ---
> src/gallium/state_trackers/clover/core/device.cpp | 11 +--
> 1 file changed, 9 insertions(+),
ce_clc_version for cl version detection instead of device_version
> Added device_clc_version in device.cpp/hpp
>
> Anyway, happy reviewing.
>
> Cc: Jan Vesely <jan.ves...@rutgers.edu>
> Cc: Pierre Moreau <pierre.mor...@free.fr>
>
signature.asc
Description: PGP signatur
On 2018-02-09 — 11:50, Aaron Watry wrote:
> No worries. I've been rebasing this series every time I've pulled
> mesa for the last few months, and this week is the first time I've had
> any real conflicts that need addressing. I'll see if I can find some
> time to address your comments and
On 2018-02-06 — 20:50, Jan Vesely wrote:
[snip]
> > > Happy to here suggestions for solving the current conflict in uses of
> > > PIPE_SHADER_CAP_PREFERRED_IR.
> >
> > One option could be to:
> > * look at the preferred IR
> > |-> if clover supports it, use it
> > |-> else, check if any IR
Acked-by: Pierre Moreau <pierre.mor...@free.fr>
On 2018-02-20 — 22:02, Karol Herbst wrote:
> not all those nir options are actually required, it just made the work a
> little easier.
>
> v2: fix asserts
> parse compute shaders
> don't lower bitfield_insert
&g
Acked-by: Pierre Moreau <pierre.mor...@free.fr>
On 2018-02-20 — 22:02, Karol Herbst wrote:
> v2: allow for non debug builds as well
> v3: move reading out env var more global
> disable tg4 with multiple offsets with nir
> disable caps for 64 bit types
>
> Sig
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3:
* Remove the const on the length argument to CreateProgramWithILKHR
(Francisco
Jerez);
* Capitalize comment (Francisco Jerez);
* Store the IL as a std::vector instead of a pointer + size (Francisco
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3: Advertise cl_khr_il_program if if the device support NATIVE as IR
src/gallium/state_trackers/clover/api/platform.cpp | 2 ++
src/gallium/state_trackers/clover/core/device.cpp | 4 +++-
2 files changed, 5 insertions
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3:
* Bump the required version of SPIRV-Tools to the latest release;
* Add a dependency on llvm-spirv.
configure.ac | 10 ++
meson.build | 4
2 files changed, 14 insertions(+)
diff --git a/config
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3: Switch from using a pointer attribute to a vector (Francisco Jerez)
src/gallium/state_trackers/clover/api/program.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/state_trackers/clov
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 3 +--
src/gallium/state_trackers/clover/api/program.cpp | 2 +-
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3: Remove the const from the length argument of clCreateProgramWithIL
src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +-
src/gallium/state_trackers/clover/api/program.cpp | 8
2 files changed, 9 inse
From: Karol Herbst <kher...@redhat.com>
v2: add more prototypes
Signed-off-by: Karol Herbst <kher...@redhat.com>
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/dispatch.cpp | 29 +++-
src/gallium/state_trackers/clover/api/
/platform.cpp”
* 15: “include/CL: Add cl_khr_il_program”
* 21: “clover: Use OpenCL 2.1 defines in place of cl_khr_il_program”
Karol Herbst (1):
clover: update ICD table to support everything up to 2.2
Pierre Moreau (20):
include/CL: Update to the latest OpenCL 2.2 headers
clover/api: Fix
If creating a library, do not allow non-compiled object in it, as
executables are not allowed, and libraries would make it really hard to
enforce the "-enable-link-options" flag.
Reviewed-by: Francisco Jerez <curroje...@riseup.net>
Signed-off-by: Pierre Moreau <pierre.mor...@
One flag that needs to be tracked is whether a library is allowed to
received mathematics optimisations or not, as the authorisation is given
when creating the library while the optimisations are specified when
creating the executable.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
--
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/program.cpp | 39 +-
src/gallium/state_trackers/clover/core/program.cpp | 3 +-
2 files changed, 25 insertions(+), 17 deletions(-)
diff --git a/src/gallium/state_trackers/
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/core/device.cpp | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/device.cpp
b/src/gallium/state_trackers/clover/core/device.cpp
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/Makefile.am | 11 +-
src/gallium/state_trackers/clover/Makefile.sources | 4 -
src/gallium/state_trackers/clover/core/program.cpp | 15 +--
src/gallium/state_trackers/clover/meson.build
Acked-by: Francisco Jerez <curroje...@riseup.net>
Reviewed-by: Karol Herbst <kher...@redhat.com>
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gall
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3: Throw an exception if the cl_khr_il_program extension is not supported
(Francisco Jerez)
src/gallium/state_trackers/clover/api/device.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/g
Reviewed-by: Francisco Jerez <curroje...@riseup.net>
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/platform.cpp | 4 ++--
src/gallium/state_trackers/clover/core/platform.cpp | 5 +
src/gallium/state_trackers/clover/core/platform.h
Reviewed-by: Karol Herbst <kher...@redhat.com>
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
include/CL/cl_ext.h | 37 +
1 file changed, 37 insertions(+)
diff --git a/include/CL/cl_ext.h b/include/CL/cl_ext.h
index 5078e8f45f..5ea49
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
.../state_trackers/clover/llvm/invocation.cpp | 26 ++
.../state_trackers/clover/llvm/invocation.hpp | 4
src/gallium/state_trackers/clover/meson.build | 2 +-
3 files changed, 31 insertions
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
Notes:
v3:
* Dropped supported_irs() (Francisco Jerez)
* Changed supports_ir() argument type to `enum pipe_shader_ir` (Francisco
Jerez)
src/gallium/state_trackers/clover/core/device.cpp | 6 ++
src/gallium/state_tr
Reviewed-by: Francisco Jerez <curroje...@riseup.net>
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 11 +--
src/gallium/state_trackers/clover/core/device.cpp | 14 ++
src/gallium/state_trackers
From the OpenCL 1.2 Specification, Section 5.6.2 (about clBuildProgram):
> If program is created with clCreateProgramWithBinary, then the
> program binary must be an executable binary (not a compiled binary or
> library).
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
-
> I think this can be merged into patches 17 and 18, no?
You are right, will do.
> On Wed, Feb 21, 2018 at 11:50 PM, Pierre Moreau <pierre.mor...@free.fr> wrote:
> > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
> > ---
> > src/gallium/state
d llvm-spirv needs to be “rebased” as some commits went
missing during the update process, but that is not going to impact the API
used; we can absolutely wait for that to finish before merging the series.
Pierre
> On Wed, Feb 21, 2018 at 11:50 PM, Pierre Moreau <pierre.mor...@free.fr> wr
On 2018-02-21 — 20:04, Aaron Watry wrote:
> On Wed, Feb 21, 2018 at 4:50 PM, Pierre Moreau <pierre.mor...@free.fr> wrote:
> > One flag that needs to be tracked is whether a library is allowed to
> > received mathematics optimisations or not, as the authorisation is gi
With the variables “dt”, “st” and “std” renamed to match existing code, this
patch is
Reviewed-by: Pierre Moreau <pierre.mor...@free.fr>
On 2018-02-20 — 22:02, Karol Herbst wrote:
> this is mostly usefull for lazy IR converters not wanting to deal with 64 bit
> lowering and other i
Acked-by: Pierre Moreau <pierre.mor...@free.fr>
On 2018-02-20 — 22:02, Karol Herbst wrote:
> this makes debugging the shader header a little easier
>
> Signed-off-by: Karol Herbst <kher...@redhat.com>
> ---
> src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 1 +
>
On 2018-02-22 — 11:04, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> >> do we already have an upstream version of both dependencies we could
> >> just use? Or do we still need special branches?
> >
> > For both we can use
On 2018-02-22 — 10:41, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
> > ---
> > src/gallium/state_trackers/clover/api/program.cpp | 39
> > +-
&
On 2018-02-22 — 11:00, Francisco Jerez wrote:
> Pierre Moreau <pierre.mor...@free.fr> writes:
>
> > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
> > ---
> > .../state_trackers/clover/llvm/invocation.cpp | 26
> >
/pierremoreau/mesa/commits/clover_spirv_series_v8
Thank you in advance,
Pierre
On 2018-05-23 — 23:58, Pierre Moreau wrote:
> Hello everyone,
>
> I updated a few patches of the series to have it use the new official
> SPIR-V <-> LLVM translator, that can be found at
&g
view: "clover: update ICD table to
> support everything up to 2.2"
>
> On Sun, Jul 15, 2018 at 9:26 PM, Pierre Moreau wrote:
> > Hello,
> >
> > I would appreciate if the following patches could be reviewed:
> > * Patch 07 (latest: v6): clover
On 2018-01-23 — 08:32, Karol Herbst wrote:
> On Tue, Jan 23, 2018 at 1:33 AM, Pierre Moreau <pierre.mor...@free.fr> wrote:
> > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
> > ---
> > src/gallium/state_trackers/clover/core/program.cpp | 22
> > +++
OpenCL 2.1, and cl_khr_il_program, gives the ability to query for a
program’s IL, which is returned as a pointer.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
.../state_trackers/clover/core/property.hpp| 39 ++
1 file changed, 39 insertions(+)
diff
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
include/CL/cl_ext.h | 34 ++
1 file changed, 34 insertions(+)
diff --git a/include/CL/cl_ext.h b/include/CL/cl_ext.h
index 710bea8837..2e4845d27d 100644
--- a/include/CL/cl_ext.h
+++ b/include/CL/cl
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
configure.ac | 5 +
meson.build | 2 ++
2 files changed, 7 insertions(+)
diff --git a/configure.ac b/configure.ac
index 7c1fbe0ed1..8c50ea6792 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2373,6 +2373,11 @@ AM_CONDI
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
.../state_trackers/clover/spirv/spirv.hpp11| 997 +
1 file changed, 997 insertions(+)
create mode 100644 src/gallium/state_trackers/clover/spirv/spirv.hpp11
diff --git a/src/gallium/state_trackers/clover
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/include/pipe/p_defines.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index b34e7a8570..082d4c4d87 100644
--- a/src/gallium/includ
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 11 +--
src/gallium/state_trackers/clover/core/device.cpp | 14 ++
src/gallium/state_trackers/clover/core/device.hpp | 1 +
3 files changed, 16 insertions(
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/platform.cpp | 4 ++--
src/gallium/state_trackers/clover/core/platform.cpp | 5 +
src/gallium/state_trackers/clover/core/platform.hpp | 2 ++
3 files changed, 9 insertions(+), 2 deletions(-)
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/api/device.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
b/src/gallium/state_trackers/clover/api/device.cpp
index 3572
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
---
src/gallium/state_trackers/clover/core/device.cpp | 11 +++
src/gallium/state_trackers/clover/core/device.hpp | 3 +++
2 files changed, 14 insertions(+)
diff --git a/src/gallium/state_trackers/clover/core/device.cpp
b/src/g
yet.
Thank you in advance for reviewing/commenting,
Pierre
[1]: https://github.com/KhronosGroup/SPIRV-Tools/
Pierre Moreau (22):
clover/api: Fix tab indentation to spaces
clover: Add additional functions to query supported IRs
clover/api: Fail if trying to build a non-executable binary
One flag that needs to be tracked is whether a library is allowed to
received mathematics optimisations or not, as the authorisation is given
when creating the library while the optimisations are specified when
creating the executable.
Signed-off-by: Pierre Moreau <pierre.mor...@free.fr>
--
101 - 200 of 309 matches
Mail list logo