Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
index 14db10b..dbf33e1 100644
--- a/
In addition, as these are never used in parallel, add a few
assertions.
v2 (Jason): Skip some complexity by putting them into a union but
pad rectangle grid into a vec4 instead. Also keep the
LOAD_UNIFORM macro.
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i9
In preparation for loading as flat vertex input.
v2: Use LOAD_INPUT() macro
Signed-off-by: Topi Pohjolainen
Reviewed-by: Jason Ekstrand (v1)
---
src/mesa/drivers/dri/i965/brw_blorp.h| 3 +--
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 24 ++--
2 files changed, 1
On 02.07.2016 16:02, Dave Airlie wrote:
> Module: Mesa
> Branch: master
> Commit: 27d456cc87a01998c6fe1dbf45937e2ca6128495
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=27d456cc87a01998c6fe1dbf45937e2ca6128495
>
> Author: Dave Airlie
> Date: Tue Jun 7 09:39:55 2016 +1000
>
> s
On Fri, Jul 01, 2016 at 01:10:59PM -0700, Jason Ekstrand wrote:
>On Thu, Jun 23, 2016 at 12:17 PM, Topi Pohjolainen
><[1]topi.pohjolai...@intel.com> wrote:
>
> Signed-off-by: Topi Pohjolainen <[2]topi.pohjolai...@intel.com>
> ---
> src/mesa/drivers/dri/i965/brw_blorp_blit.c
On Fri, Jul 01, 2016 at 01:03:08PM -0700, Jason Ekstrand wrote:
>Since we're using inputs_read and the packing from the fs compiler, is
>this really needed? It seems like we could just add two floats worth
>of padding after rect_grid_y1 to pad it to a vec4.
Good call. I just revised i
https://bugs.freedesktop.org/show_bug.cgi?id=90264
Jeremy Nickurak changed:
What|Removed |Added
CC||freedesktop-bugs@trk.nickur
The extension is already exposed, this simply marks it as done.
Signed-off-by: Lars Hamre
---
Note: someone with access will need to commit this after the review process
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index ce34869..
On Sun, 2016-07-03 at 17:51 -0700, Francisco Jerez wrote:
> Reviewed-by: Serge Martin
> ---
> .../state_trackers/clover/llvm/invocation.cpp | 223
> ++---
> 1 file changed, 103 insertions(+), 120 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/llvm/invocatio
On Sun, 2016-07-03 at 17:51 -0700, Francisco Jerez wrote:
> Some assorted and mostly trivial clean-ups for the source to bitcode
> compilation path.
>
> Reviewed-by: Serge Martin
> ---
> .../state_trackers/clover/llvm/invocation.cpp | 52
> +-
> 1 file changed, 22 inser
Split the work previously done by compile_program_llvm() into
compile_program() (which simply runs the front-end and serializes the
resulting LLVM IR) and link_program() (which takes care of everything
else down to binary codegen).
[ Serge Martin: allow LLVM IR dump after compilation ]
Reviewed-b
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 26 +++---
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index e9b3db
Most significant change is debugging flags are now a scoped enum and
all debugging helpers live in the debug namespace.
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 81 +++---
1 file changed, 41 insertions(+), 40 deletions(-)
diff --git a/sr
This partially reverts 7e0180d57d330bd8d3047e841086712376b2a1cc.
Having two different exception subclasses for compilation and linking
makes it more difficult to share or move code between the two
codepaths, because the exact same function under the same error
condition would need to throw one exce
From: Serge Martin
[ Francisco Jerez: Use validate_build_common for error checking,
simplify control flow slightly and handle additional exception
types. ]
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/program.cpp | 31 ---
1 file changed, 27 in
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 14 +-
1 file changed, 5 insertions(+), 9 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 2ca57d0..962f
This allows simplifying the interface of compile_llvm() because it no
longer needs to read out and return the optimization level and address
space map from the compiler instance. Instead declare the compiler
instance at the top level so that both properties are available
directly.
Reviewed-by: Se
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 3 +-
.../state_trackers/clover/core/compiler.hpp| 3 --
src/gallium/state_trackers/clover/core/program.cpp | 3 +-
.../state_trackers/clover/tgsi/compiler.cpp| 5 ++--
.../state_trackers/clover
From: Serge Martin
Reviewed-by: Francisco Jerez
---
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/ga
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 1 +
src/gallium/state_trackers/clover/llvm/codegen.hpp | 9 ++
.../state_trackers/clover/llvm/codegen/native.cpp | 165 +
.../state_trackers/clover/llvm/invocation.cpp | 119 -
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 56 +++---
1 file changed, 29 insertions(+), 27 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 962f1b
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/api/program.cpp | 33 +--
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state_trackers/clover/api/program.cpp
index 0303c1e..c3f9cb
Some of these will be useful from a different compilation unit in the
same subtree so put them in a publicly accessible header file.
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 3 +-
.../state_trackers/clover/llvm/invocation.cpp | 61 +-
s
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/tgsi/compiler.cpp | 7 +++
src/gallium/state_trackers/clover/tgsi/invocation.hpp | 2 ++
2 files changed, 9 insertions(+)
diff --git a/src/gallium/state_trackers/clover/tgsi/compiler.cpp
b/src/gallium/state_trackers/clover/tg
This is the common part of the code used to generate a clover::module
from LLVM bitcode, shared between the native and LLVM paths.
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 2 +
src/gallium/state_trackers/clover/llvm/codegen.hpp | 48 +
.../state_t
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 25 +++---
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 7cfd44
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/llvm/compat.hpp | 19 +++
src/gallium/state_trackers/clover/llvm/invocation.cpp | 19 ---
2 files changed, 35 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/comp
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 97 ++
1 file changed, 5 insertions(+), 92 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 0fc6190
From: Serge Martin
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 | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/gallium/s
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 34 ++
1 file changed, 3 insertions(+), 31 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index afe621d
Superseded by compile_program() and link_program().
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/core/compiler.hpp | 7 ---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 11 ---
2 files changed, 18 deletions(-)
diff --git a/src/gallium/state_trackers/
Drop a few include and using directives which are no longer necessary.
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 57 +++---
1 file changed, 7 insertions(+), 50 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cp
This function was doing three separate things:
- Initializing and releasing the ELF parsing state (the latter can be
better done using RAII).
- Searching for the symbol table in the ELF file.
- Extraction of kernel symbol offsets from the symbol table.
Split each one into a separate function
header_map was the only definition left in compiler.hpp, move it into
program.hpp which is its only user in clover/core.
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 1 -
.../state_trackers/clover/core/compiler.hpp| 34 --
src/ga
So it can be shared between the compilation and linking codepaths.
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 83 +++---
1 file changed, 43 insertions(+), 40 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/api/program.cpp | 6 --
src/gallium/state_trackers/clover/core/error.hpp | 6 ++
src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +-
3 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/src/galli
[ Serge Martin: disable internalize pass when building a library.
Otherwise some functions may be inlined and removed ]
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 47 +++---
.../state_trackers/clover/util/functional.hpp | 21
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 38 --
1 file changed, 20 insertions(+), 18 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index f0f1b7
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 1 +
src/gallium/state_trackers/clover/llvm/codegen.hpp | 7 ++
.../state_trackers/clover/llvm/codegen/bitcode.cpp | 76 ++
.../state_trackers/clover/llvm/invocation.cpp | 35 --
From: Serge Martin
Reviewed-by: Francisco Jerez
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index 8321e8e..c5e1d55 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2182,7 +2182,7 @@ if test "x$enable_gallium_llvm" = xyes; then
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 45 --
1 file changed, 24 insertions(+), 21 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 2f5c8a
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/llvm/codegen.hpp | 7 ++
.../state_trackers/clover/llvm/codegen/bitcode.cpp | 25 +-
2 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/llvm/codegen.hpp
b/src/ga
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 88 ++
1 file changed, 41 insertions(+), 47 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 1e4bed
This gets rid of the program::build_* query methods and replaces them
with the program::build() method that returns a single data structure
containing all parameters for the last build done on the given target
device (including build logs, options and the binary itself).
[ Serge Martin: Fix invert
[ Serge Martin: Fix inverted opts and log build ctor args.
Keep the log related to the build. Fix indentation ]
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/api/program.cpp | 8 +++--
src/gallium/state_trackers/clover/core/program.cpp | 34 ++
src/galli
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 36 --
1 file changed, 20 insertions(+), 16 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 21e6e0
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 17 ++---
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index d4c8add..
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/api/program.cpp | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/gallium/state_trackers/clover/api/program.cpp
index 2dc9ee7..1836099 10064
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 223 ++---
1 file changed, 103 insertions(+), 120 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 754e
---
src/gallium/state_trackers/clover/Makefile.sources | 1 +
.../state_trackers/clover/llvm/metadata.hpp| 132 +
2 files changed, 133 insertions(+)
create mode 100644 src/gallium/state_trackers/clover/llvm/metadata.hpp
diff --git a/src/gallium/state_trackers/clover
This switches compile_native() to the C++ API (which the rest of this
file makes use of anyway so there is little benefit from using the C
API), what should get rid of an amount of boilerplate and fix a leak
of the TargetMachine object in the error path.
v2: Additional fixes for LLVM 3.6.
v3: Upda
This gets rid of most ifdef's from the invocation.cpp code -- Only a
couple of them are left which will be removed differently in the
following commits.
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/Makefile.sources | 1 +
src/gallium/state_trackers/clover/llvm/compat.hpp |
This ifdef'ed out code was meant to handle compilation into TGSI, but
it doesn't seem likely that it will ever be useful even if the TGSI
back-end is resurrected because the TGSI bitcode can just be plumbed
through in ELF format and dealt with as a regular "native" back-end.
Reviewed-by: Serge Mar
This series is the result of a long back and forth of patches between
Serge and me. Most of the changes are already reviewed by either of
us, but I'm sending them again over the mailing list just in case
somebody else wants to give additional feedback. Patches 1-30 are
mainly cleaning up and rewo
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 40 +++---
1 file changed, 4 insertions(+), 36 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 44c7e1e
This will be shared between the compiler and linker codepaths.
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 99 ++
1 file changed, 47 insertions(+), 52 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/sr
Some assorted and mostly trivial clean-ups for the source to bitcode
compilation path.
Reviewed-by: Serge Martin
---
.../state_trackers/clover/llvm/invocation.cpp | 52 +-
1 file changed, 22 insertions(+), 30 deletions(-)
diff --git a/src/gallium/state_trackers/clover/l
Reviewed-by: Serge Martin
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 242c096..754e477
From: Francesco Ansanelli
---
src/gallium/drivers/r600/evergreen_compute.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/evergreen_compute.c
b/src/gallium/drivers/r600/evergreen_compute.c
index 8db525e..6f39c90 100644
--- a/src/gallium/drivers/r
The call to _mesa_update_shader_textures_used() already takes
care of copying for us.
---
src/mesa/main/uniform_query.cpp | 4
1 file changed, 4 deletions(-)
diff --git a/src/mesa/main/uniform_query.cpp b/src/mesa/main/uniform_query.cpp
index 67375a1..f2a9d3d 100644
--- a/src/mesa/main/unifo
60 matches
Mail list logo