[Mesa-dev] [Bug 101971] Mesa fails to build when Wayland and EGL support enabled due to missing C++ lib from mesautil

https://bugs.freedesktop.org/show_bug.cgi?id=101971 Mike Lothian changed: What|Removed |Added CC||m...@fireburn.co.uk

[Mesa-dev] [PATCH 1/3] glsl: calculate number of operands in an expression once

Extra validation is added to ir_validate to make sure this is always updated to the correct numer of operands, as passes like lower_instructions modify the instructions directly rather then generating a new one. --- src/compiler/glsl/glsl_to_nir.cpp | 4 +--

[Mesa-dev] [PATCH 2/3] glsl: tidy up get_num_operands()

Also add a comment that this should only be used by the ir_reader interface for testing purposes. --- src/compiler/glsl/ir.cpp | 8 ++-- src/compiler/glsl/ir.h | 14 +++--- 2 files changed, 9 insertions(+), 13 deletions(-) diff --git a/src/compiler/glsl/ir.cpp

[Mesa-dev] [PATCH 3/3] glsl: stop copying struct and interface member names

We are currently copying the name for each member dereference but we can just share a single instance of the string provided by the type. This change also stops us recalculating the field index repeatedly. --- src/compiler/glsl/ast_array_index.cpp | 14 -

Re: [Mesa-dev] [PATCH] i965/drm: Inline brw_bo_references.

Rb On July 30, 2017 4:08:48 PM Kenneth Graunke wrote: It's a single atomic add, so it makes sense to inline it. Improves performance in Piglit's drawoverhead microbenchmark's "DrawArrays ( 1 VBO, 0 UBO, 0) w/ no state change" subtest by 0.400922% +/- 0.310389%

Re: [Mesa-dev] [PATCH 2/3] glsl: tidy up get_num_operands()

On 31/07/17 08:51, Ilia Mirkin wrote: On Jul 30, 2017 5:38 PM, "Timothy Arceri" > wrote: Also add a comment that this should only be used by the ir_reader interface for testing purposes. --- src/compiler/glsl/ir.cpp | 8

Re: [Mesa-dev] [PATCH v2] st/dri: add 32-bit RGBX/RGBA formats

On Sun, Jul 30, 2017 at 5:49 PM, Chih-Wei Huang wrote: > 2017-07-28 22:59 GMT+08:00 Marek Olšák : > > Hi, > > > > I've sent a request to revert this commit in 17.2. I'll keep it in > > master, but I'll add a fix not to expose the new formats for GLX. >

[Mesa-dev] [PATCH] radv: Don't underflow non-visible VRAM size.

n some APU situations the reported visible size can be larger than VRAM size. This properly clamps the value. Surprisingly both CTS and spec seem to allow a heap type with size 0, so this seemed like the easiest option to me. Signed-off-by: Bas Nieuwenhuizen Fixes: 4ae84efbc5c

Re: [Mesa-dev] [PATCH 2/3] glsl: tidy up get_num_operands()

On Jul 30, 2017 5:38 PM, "Timothy Arceri" wrote: Also add a comment that this should only be used by the ir_reader interface for testing purposes. --- src/compiler/glsl/ir.cpp | 8 ++-- src/compiler/glsl/ir.h | 14 +++--- 2 files changed, 9 insertions(+),

[Mesa-dev] [PATCH] i965/drm: Inline brw_bo_references.

It's a single atomic add, so it makes sense to inline it. Improves performance in Piglit's drawoverhead microbenchmark's "DrawArrays ( 1 VBO, 0 UBO, 0) w/ no state change" subtest by 0.400922% +/- 0.310389% (n=350) on my i7-7700HQ. --- src/mesa/drivers/dri/i965/brw_bufmgr.c | 6 --

Re: [Mesa-dev] [PATCH] st/dri: enable 32-bit RGBX/RGBA formats only on Android

On Sun, Jul 30, 2017 at 4:44 AM, Tomasz Figa wrote: > On Sun, Jul 30, 2017 at 6:51 AM, Marek Olšák wrote: > > Hi Tomasz, > > > > This is for 17.2 only. Not master. I have a different fix for master. > > Right, we don't need this in master, but according to

Re: [Mesa-dev] [PATCH v2] st/dri: add 32-bit RGBX/RGBA formats

Hi Rob, Sorry to bother you again. This patch also breaks the srwast/llvmpipe on Android since the red and blue are just swapped. I guess nouveau has the same issue but I'm waiting Mauro's confirmation. Any comment? -- Chih-Wei Android-x86 project http://www.android-x86.org

Re: [Mesa-dev] [PATCH v2] st/dri: add 32-bit RGBX/RGBA formats

2017-07-28 22:59 GMT+08:00 Marek Olšák : > Hi, > > I've sent a request to revert this commit in 17.2. I'll keep it in > master, but I'll add a fix not to expose the new formats for GLX. I thought it doesn't break GLX according to the commit message (investigation by Chad

Re: [Mesa-dev] [PATCH] nv50/ir: fix MUL ConstantFolding with saturation

Looks good to me! Reviewed-by: Tobias Klausmann On 7/30/17 5:51 PM, Karol Herbst wrote: > For mul(a, +-1) codegen could end up generating OP_MOV with a saturation flag > set which we got simply ignored. > > Adding an assert for detecting more of such

[Mesa-dev] [PATCH] nv50/ir: fix MUL ConstantFolding with saturation

For mul(a, +-1) codegen could end up generating OP_MOV with a saturation flag set which we got simply ignored. Adding an assert for detecting more of such issues. Fixes wrongly rendered water in Hitman Absolution running under wine. Also a few shaders in Mad Max and Alien Isolation produce such

[Mesa-dev] [PATCH 4/8] clover/llvm: Use -cl-std and device version to select language defaults

According to section 5.8.4.5 of the 2.0 spec, the CL C version is chosen by: 1) If you have -cl-std=CL1.1+ use the version specified 2) If not, use the highest 1.x version that the device supports Curiously, there is no valid value for -cl-std=CL1.0 Signed-off-by: Aaron Watry

[Mesa-dev] [PATCH 3/8] clover: Add device_clc_version to llvm::[compile|link]_program

We'll be using it to select the default language version soon. Signed-off-by: Aaron Watry Cc: Pierre Moreau Cc: Jan Vesely v2: (Pierre) Move changes to create_compiler_instance invocation to correct patch to prevent

[Mesa-dev] [PATCH 5/8] clover/llvm: Use device in llvm compilation instead of copying fields

Copying the individual fields from the device when compiling/linking will lead to an unnecessarily large number of fields getting passed around. Signed-off-by: Aaron Watry Cc: Jan Vesey --- src/gallium/state_trackers/clover/core/program.cpp | 9

[Mesa-dev] [PATCH 8/8] clover/llvm: Make __OPENCL_VERSION__ dynamic

Signed-off-by: Aaron Watry CC: Jan Vesely v2: base it on the device version --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 7/8] clover/llvm: validate requested cl-std against device_clc_version

Signed-off-by: Aaron Watry Cc: Pierre Moreau --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp

[Mesa-dev] [PATCH 2/8] clover: Add device_clc_version to device.[hc]pp

device_version and device_clc_version are not necessarily the same for devices that support CL 1.0, but have a 1.1 compiler and the necessary extensions. CC: Jan Vesey --- src/gallium/state_trackers/clover/api/device.cpp | 2 +-

[Mesa-dev] [PATCH 6/8] clover/llvm: Pass device down to compile

We'll need to be able to detect device version to define the appropriate __OPENCL_VERSION__ header. Signed-off-by: Aaron Watry --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 1/8] clover/device: Move device version into core/device.cpp

The device version is the maximum CL version that the device supports. Eventually, this will be based on the features/extensions of the actual device, but for now move it a bit closer to its eventual destination. Signed-off-by: Aaron Watry ---

[Mesa-dev] [PATCH 0/8 v2] A few clover fixes for both CTS and eventual 1.2 support

I've dropped the first patch of the previous series for now. I'm not withdrawing it completely, just going to see if there's anything about the user_ptr stuff that could have been causing the issue instead, and if I'm using too big a hammer in this patch. If I convince myself of its correctness,

Re: [Mesa-dev] [PATCH 1/1] gallium, clover: Wait for requested operation if blocking flag is set

What's here is: Reviewed-By: Aaron Watry That being said, are clEnqueueMapBuffer/clEnqueueMapImage still affected or are those always done as blocking operations and my c++ is weak? --Aaron On Fri, Jul 28, 2017 at 10:36 PM, Jan Vesely wrote: >

Re: [Mesa-dev] [PATCH] radv: Don't underflow non-visible VRAM size.

On 31/07/17 06:29 AM, Bas Nieuwenhuizen wrote: > n some APU situations the reported visible size can be larger than Missing "I" at the beginning, vim fail? :) > VRAM size. This properly clamps the value. > > Surprisingly both CTS and spec seem to allow a heap type with size 0, > so this seemed

Re: [Mesa-dev] i965: Transition the rest of surfaces (i.e., color) to isl

On 07/29/2017 08:21 AM, Jason Ekstrand wrote: On Fri, Jul 28, 2017 at 10:14 PM, Tomasz Figa > wrote: On Sat, Jul 29, 2017 at 2:06 PM, Jason Ekstrand > wrote: > On Fri, Jul 28, 2017