On 20.04.2016 02:52, Tom Stellard wrote:
> The range metadata tells LLVM the range of expected values for this intrinsic,
> so it can do some additional optimizations on the result.
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 29 ++---
> 1 file changed, 26 insertions
On 20.04.2016 03:39, Emil Velikov wrote:
> On 19 April 2016 at 15:47, Chuck Atkins wrote:
>> This still doesn't quite give what you want. One can also have an llvm with
>> component shared libs. So there's three different options for llvm library
>> configurations: a single shared lib, component
https://bugs.freedesktop.org/show_bug.cgi?id=95003
--- Comment #3 from Serge Martin ---
That would be CL_KERNEL_PREFERRED_WORK_GROUP_SIZE_MULTIPLE from
clGetKernelWorkGroupInfo
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug._
On Wed, 2016-04-20 at 08:37 +0200, Iago Toral wrote:
> On Tue, 2016-04-19 at 15:32 -0700, Jason Ekstrand wrote:
> >
> >
> > On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez
> > wrote:
> > From: Iago Toral Quiroga
> >
> > At least i965 hardware does not have na
On 20.04.2016 03:13, Oded Gabbay wrote:
> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
>> On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote:
>>> On Mon, Apr 18, 2016 at 6:03 PM,
>>> Ilia Mirkin wrote:
On Mon, Apr 18, 2016 at 10:47 AM, Oded Gabbay
wrote:
> On Thu, Apr 14,
In order to implement the "uninitialized variable" warning it was
needed to hande several corner cases manually. Recently I sent a series
to fix the last false positive:
https://lists.freedesktop.org/archives/mesa-dev/2016-April/113614.html
At this point I think that it would be better to include
---
Those are basically the shaders I used while developing the warning,
plus some cleaning and some extra ones.
The only reason they are vertex shaders is that the standalone compiler
needs to receive a shader with a valid shader extension.
.../glsl/tests/warnings/000-basic-test.vert|
Add an option in order to ask to just print the InfoLog, without any
header or separator. Useful if we want to use the standalone compiler
to track only the warning/error messages.
---
src/compiler/glsl/main.cpp | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff --gi
It executes compiler-glsl on all the available shaders, and it checks
that the outcome is the expected.
Bash code based on the already existing optimization-test
---
src/compiler/Makefile.glsl.am | 3 ++-
src/compiler/glsl/tests/warnings-test | 31 +++
2 files
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #5 from Timo Aaltonen ---
llvm-3.8 misdetects skylake features, this is fixed in 3.9-snapshot..
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
> On 20.04.2016 03:13, Oded Gabbay wrote:
>> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
>>> On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote:
On Mon, Apr 18, 2016 at 6:03 PM,
Ilia Mirkin wrote:
> On Mon, Apr 18, 201
On 19.04.2016 00:55, Nicolas Dufresne wrote:
> Le lundi 18 avril 2016 à 11:40 +0900, Michel Dänzer a écrit :
>> On 17.04.2016 09:49, nico...@ndufresne.ca wrote:
>>>
>>> From: Nicolas Dufresne
>>>
>>> Sorry for the long delay breaking down this patch. I have now
>>> rebased
>>> on top recent mesa t
On 20.04.2016 17:48, Oded Gabbay wrote:
> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
>> On 20.04.2016 03:13, Oded Gabbay wrote:
>>> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote:
> On Mon, Apr 18, 2016 at 6:03 PM,
On Wed, Apr 20, 2016 at 12:04 PM, Michel Dänzer wrote:
> On 20.04.2016 17:48, Oded Gabbay wrote:
>> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
>>> On 20.04.2016 03:13, Oded Gabbay wrote:
On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
> On Tue, Apr 19, 2016 at 3:11 PM, O
On Wed, Apr 20, 2016 at 11:14 AM, Oded Gabbay wrote:
> On Wed, Apr 20, 2016 at 12:04 PM, Michel Dänzer wrote:
>> On 20.04.2016 17:48, Oded Gabbay wrote:
>>> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
On 20.04.2016 03:13, Oded Gabbay wrote:
> On Tue, Apr 19, 2016 at 5:59 PM,
On 20 April 2016 at 01:46, Elie TOURNIER wrote:
> Now Windows users have the same doxygen files than *nix users
> Not tested (I don't have a windows)
>
> Signed-off-by: Elie TOURNIER
> ---
> doxygen/doxy.bat | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git a/doxygen/doxy.bat b/doxy
For the series:
Reviewed-by: Marek Olšák
You might want to wait for others to comment.
Mare
On Tue, Apr 19, 2016 at 9:38 PM, Rob Herring wrote:
> Add MESA_FORMAT_R8G8B8A8_UNORM and MESA_FORMAT_R8G8B8X8_UNORM formats as
> these are the preferred formats for Android.
>
> Signed-off-by: Rob Herr
According to the comments in src/glx/glxcurrent.c __glXGetCurrentContext()
always returns a valid pointer. If no context is made current, it will
contain dummyContext. Thus a test for NULL will always fail.
Patch author: Egbert Eich
https://bugzilla.opensuse.org/show_bug.cgi?id=962609
Signed-of
Removes the need to set LIBVA_DRIVER_NAME=gallium for supported targets and is
consistent with vdpau and general gallium drivers.
Author: Jimmy Berry
Signed-off-by: Jimmy Berry
---
src/gallium/targets/va/Makefile.am | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/galliu
On Wed, Apr 20, 2016 at 1:29 AM, Bas Nieuwenhuizen
wrote:
> I retract patch 1 and 2. Large scratch buffers are nice, but the
> hardware only supports a 32-bit offset into it.
You can still allocate a smaller scratch buffer. This should limit the
number of waves in hw. TMPRING_SIZE.WAVES should be
Patches 1-3:
Reviewed-by: Marek Olšák
Patch 4:
Acked-by: Marek Olšák
Marek
On Tue, Apr 19, 2016 at 7:52 PM, Tom Stellard wrote:
> The ds_bpermute instruction allows threads to transfer data directly
> to or from the vgprs of other threads. These instructions use the lds
> hardware to transf
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #6 from Jose Fonseca ---
It's not the first time LLVM misidentifies modern CPUs.
I thought that all the logic in src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
for setting +/-foo mattrs would save us from this sort of grief.
On the o
Hi,
On 15-04-16 00:29, Samuel Pitoiset wrote:
On 04/12/2016 12:04 PM, Hans de Goede wrote:
Hi,
On 08-04-16 18:14, Samuel Pitoiset wrote:
On 04/08/2016 12:17 PM, Hans de Goede wrote:
Hi,
On 23-03-16 23:10, Samuel Pitoiset wrote:
Are you sure this won't break compute shaders on fermi?
Co
On Tue, Apr 19, 2016 at 11:02 PM, Rob Herring wrote:
>> It would ofc be good to make sure we don't break things for
>> android-x86, but I think they would stick on dri2 + pre-atomic state
>> w/ compile time build decisions until enough of the desktop gpu's
>> support atomic.. although maybe it hel
On 20 April 2016 at 03:27, Jonathan Gray wrote:
> On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote:
>> On 19 April 2016 at 04:03, Jonathan Gray wrote:
>> > Any objections to this?
>> >
>> Absolutely none. I simply missed it.
>>
>> > On Mon, Dec 21, 2015 at 04:39:55PM +1100, Jonathan G
On 19/04/16 19:39, Emil Velikov wrote:
Hi Chuck,
Thanks for chipping in.
On 19 April 2016 at 15:47, Chuck Atkins wrote:
This still doesn't quite give what you want. One can also have an llvm with
component shared libs. So there's three different options for llvm library
configurations: a si
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #7 from Roland Scheidegger ---
(In reply to Jose Fonseca from comment #6)
> It's not the first time LLVM misidentifies modern CPUs.
>
> I thought that all the logic in
> src/gallium/auxiliary/gallivm/lp_bld_misc.cpp for setting +/-fo
Some CAPs are stored as 64-bit value while Mesa stores
the related constant as 32-bit value.
Signed-off-by: Bas Nieuwenhuizen
---
src/mesa/state_tracker/st_extensions.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/
On Tue, Apr 19, 2016 at 8:45 PM, Mauro Rossi wrote:
> LOCAL_C_INCLUDES needs an additional path to fix the following build error:
>
> external/mesa/src/compiler/glsl/glcpp/glcpp-lex.l:30:25: fatal error:
> glcpp-parse.h: No such file or directory
> #include "glcpp-parse.h"
>
The mode should stay the same as the original struct. In
particular, shared should not be changed to temporary.
Signed-off-by: Bas Nieuwenhuizen
---
src/compiler/glsl/opt_structure_splitting.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/compiler/glsl/opt_structure_s
On Wed, Apr 20, 2016 at 01:33:24PM +0100, Emil Velikov wrote:
> On 20 April 2016 at 03:27, Jonathan Gray wrote:
> > On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote:
> >> On 19 April 2016 at 04:03, Jonathan Gray wrote:
> >> > Any objections to this?
> >> >
> >> Absolutely none. I simp
While i never got this error in the first place, I can attest that it
certainly hasn't broken any use in paraview, so...
Tested-by: Chuck Atkins
- Chuck
On Tue, Apr 19, 2016 at 1:45 AM, Markus Wick wrote:
> Am 2016-04-19 01:12, schrieb Tim Rowley:
>
>> Code was using an incorrect address for
On 19 April 2016 at 20:59, Tom Stellard wrote:
> On Tue, Apr 19, 2016 at 07:39:13PM +0100, Emil Velikov wrote:
>> Hi Chuck,
>>
>> Thanks for chipping in.
>>
>> On 19 April 2016 at 15:47, Chuck Atkins wrote:
>> > This still doesn't quite give what you want. One can also have an llvm
>> > with
>>
Patched 3-5 are:
Reviewed-by: Iago Toral Quiroga
On Mon, 2016-04-18 at 19:04 -0700, Jason Ekstrand wrote:
> We shouldn't be reading the const_index directly
> ---
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drive
2016-04-20 3:38 GMT+08:00 Rob Herring :
> The RGBX/RGBA pixel formats used in the Android EGL don't get configs
> created due to the missing formats in the DRI state tracker. This series
> adds the necessary formats for configs and DRI images. Support in GBM is
> also added as it will be needed soo
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 20, 2016 at 3:35 PM, Bas Nieuwenhuizen
wrote:
> Some CAPs are stored as 64-bit value while Mesa stores
> the related constant as 32-bit value.
>
> Signed-off-by: Bas Nieuwenhuizen
> ---
> src/mesa/state_tracker/st_extensions.c | 8 ++--
> 1 file
Hi Marek,
This patch ended up breaking make check for me (src/egl/egl-symbols-check).
Also, while bisecting, it occurred to me that you pushed v2 of the series,
while you had previously sent a v3 to the mesa-dev list. Was pushing v2
intentional?
--Aaron
On Tue, Mar 8, 2016 at 4:52 PM, Marek Olš
On Wed, Apr 20, 2016 at 4:56 PM, Aaron Watry wrote:
> Hi Marek,
>
> This patch ended up breaking make check for me (src/egl/egl-symbols-check).
>
> Also, while bisecting, it occurred to me that you pushed v2 of the series,
> while you had previously sent a v3 to the mesa-dev list. Was pushing v2
>
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #8 from Timo Aaltonen ---
Actually it wasn't avx512, that was the first one I tried :) It's enabled also
on 3.7 and that version works fine. Only one that was added in 3.8 is PKU, but
dropping just that didn't help.
I did try droppin
Reviewed-by: Nicolai Hähnle
On 20.04.2016 08:35, Bas Nieuwenhuizen wrote:
Some CAPs are stored as 64-bit value while Mesa stores
the related constant as 32-bit value.
Signed-off-by: Bas Nieuwenhuizen
---
src/mesa/state_tracker/st_extensions.c | 8 ++--
1 file changed, 6 insertions(+),
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
src/mesa/state_tracker/st_cb_queryobj.c | 55 -
1 file changed, 33 insertions(+), 22 deletions(-)
diff --git a/src/mesa/state_tracker/st
From: Nicolai Hähnle
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94984
---
src/gallium/drivers/radeon/r600_query.c | 40 ++---
src/gallium/drivers/radeon/r600_query.h | 2 +-
2 files changed, 33 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drive
From: Marek Olšák
- use an enum
- use a unique slot number regardless of the shader stage
(the per-stage slots will go away for RW buffers)
---
src/gallium/drivers/radeonsi/si_descriptors.c | 6 ++---
src/gallium/drivers/radeonsi/si_shader.c| 13 ++
src/gallium/drivers/radeo
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
src/gallium/drivers/radeonsi/si_state.c | 4 ++--
src/gallium/drivers/radeonsi/si_state.h | 1 +
3 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drive
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_descriptors.c | 50 +--
src/gallium/drivers/radeonsi/si_pipe.h| 2 +-
2 files changed, 25 insertions(+), 27 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/gallium/drivers/radeo
Hi,
This moves all private shader resources to the RW buffer bindings, including
all driver constant buffers, and the poly stipple image (which is converted
into a constant buffer).
RW buffer descriptors are made global, not per shader stage, so all shaders
receive the same pointer.
Finally,
From: Marek Olšák
this will be used in the next commit
---
src/gallium/drivers/radeonsi/si_descriptors.c | 27 +--
1 file changed, 17 insertions(+), 10 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/gallium/drivers/radeonsi/si_descriptors.
From: Marek Olšák
const buffers are no longer used since the clip plane const buffer was
moved to RW buffers
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/
From: Marek Olšák
add it to the RW_BUFFERS descriptor array
now the slot masks don't have to have 64 bits
---
src/gallium/drivers/radeonsi/si_descriptors.c | 23 +
src/gallium/drivers/radeonsi/si_pipe.c| 2 -
src/gallium/drivers/radeonsi/si_pipe.h| 1 -
src/gallium/dri
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c| 19 +++
src/gallium/drivers/radeonsi/si_shader.h| 6 ++
src/gallium/drivers/radeonsi/si_state.c | 4 ++--
src/gallium/drivers/radeonsi/si_state.h | 1 +
src/gallium/drivers/rade
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_descriptors.c | 14 +++---
src/gallium/drivers/radeonsi/si_shader.c | 5 +++--
src/gallium/drivers/radeonsi/si_state.c | 3 ++-
src/gallium/drivers/radeonsi/si_state.h | 4
4 files changed, 12 insertions(+), 1
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_blit.c| 12 +--
src/gallium/drivers/radeonsi/si_descriptors.c | 110 +-
src/gallium/drivers/radeonsi/si_pipe.h| 4 +-
src/gallium/drivers/radeonsi/si_state.h | 4 +-
4 files changed, 64 inser
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.c | 6 +++---
src/gallium/drivers/radeonsi/si_state.h | 25 +++--
3 files changed, 12 insertions(+), 23 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/s
From: Nicolai Hähnle
Even when begin_query succeeds, there can still be failures in query handling.
For example for radeon, additional buffers may have to be allocated when
queries span multiple command buffers.
---
src/gallium/drivers/ddebug/dd_context.c | 4 ++--
src/gallium/drivers/fr
From: Nicolai Hähnle
---
src/gallium/drivers/radeon/r600_query.c | 13 -
src/gallium/drivers/radeon/r600_query.h | 4 ++--
2 files changed, 10 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_query.c
b/src/gallium/drivers/radeon/r600_query.c
index 813885b
This header should not blindly include the GLX and should also
conditionally define functions that use GLX parameters. The
MESA_EGL_NO_X11_HEADERS macro is used to check if we're building without
GLX support.
Signed-off-by: Mircea Gherzan
---
include/GL/mesa_glinterop.h | 37
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #9 from Roland Scheidegger ---
(In reply to Timo Aaltonen from comment #8)
> Actually it wasn't avx512, that was the first one I tried :) It's enabled
> also on 3.7 and that version works fine. Only one that was added in 3.8 is
> PKU,
Reviewed-by: Samuel Pitoiset
On 04/20/2016 03:35 PM, Bas Nieuwenhuizen wrote:
Some CAPs are stored as 64-bit value while Mesa stores
the related constant as 32-bit value.
Signed-off-by: Bas Nieuwenhuizen
---
src/mesa/state_tracker/st_extensions.c | 8 ++--
1 file changed, 6 insertions(
Yeah, this makes sense.
Reviewed-by: Samuel Pitoiset
On 04/20/2016 05:43 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Even when begin_query succeeds, there can still be failures in query handling.
For example for radeon, additional buffers may have to be allocated when
queries span multipl
On 04/20/2016 05:43 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
src/mesa/state_tracker/st_cb_queryobj.c | 55 -
1 file changed, 33 insertions(+), 22
Rob Herring writes:
> The RGBX/RGBA pixel formats used in the Android EGL don't get configs
> created due to the missing formats in the DRI state tracker. This series
> adds the necessary formats for configs and DRI images. Support in GBM is
> also added as it will be needed soon for Android.
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 20, 2016 at 5:52 PM, Mircea Gherzan
wrote:
> This header should not blindly include the GLX and should also
> conditionally define functions that use GLX parameters. The
> MESA_EGL_NO_X11_HEADERS macro is used to check if we're building without
> GLX s
Minor comment below. Either way:
Reviewed-by: Sinclair Yeh
On Wed, Apr 20, 2016 at 05:52:17PM +0200, Mircea Gherzan wrote:
> This header should not blindly include the GLX and should also
> conditionally define functions that use GLX parameters. The
> MESA_EGL_NO_X11_HEADERS macro is used to ch
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.c
Forcing the destination type to be signed when the source is signed
is not totally correct.
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/no
https://bugs.freedesktop.org/show_bug.cgi?id=95035
Bug ID: 95035
Summary: Gallium OSMesa driver is far from being thread-safe
Product: Mesa
Version: 11.2
Hardware: Other
OS: All
Status: NEW
Severity: norma
For patches 1,3,4:
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 20, 2016 at 5:43 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94984
> ---
> src/gallium/drivers/radeon/r600_query.c | 40
> ++---
> src/ga
On Thu, Apr 14, 2016 at 9:29 AM, Michel Dänzer wrote:
> On 14.04.2016 11:37, Michel Dänzer wrote:
>> On 12.04.2016 21:33, Marek =?UNKNOWN?B?T2zFocOhaw==?= wrote:
>>>
>>> URL:
>>> http://cgit.freedesktop.org/mesa/mesa/commit/?id=5a4b74d1ba2c156766a7a5dbfef099c7db5d6694
>>> Author: Marek Olšák
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 1:06 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> Cc: "11.1 11.2"
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/n
Presumably you'd want to touch up the non-limm side of this as well?
Although TBH, I can't really think of a time when it'd matter. I think
you're pretty much guaranteed to have stype == dtype for mul's. Maybe
I'm wrong?
-ilia
On Wed, Apr 20, 2016 at 1:06 PM, Samuel Pitoiset
wrote:
> Forcing t
This is only valid for other atomic operations (including CAS). This
fixes an invalid opcode error from dmesg. While we are it, make sure
to initialize global addr to 0 for other atomic operations.
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110
The SM20/SM30 logic does this for exch || cas. Are you *sure* this
shouldn't be the same? It's pretty silly to do a CAS without a dst,
but it's definitely possible (through DCE).
On Wed, Apr 20, 2016 at 1:47 PM, Samuel Pitoiset
wrote:
> This is only valid for other atomic operations (including CA
Pushed!
On Sat, Apr 16, 2016 at 9:52 PM, Jason Ekstrand
wrote:
> Reviewed-by: Jason Ekstrand
> On Apr 16, 2016 12:50 PM, "Laurent Carlier" wrote:
>
>> mcpu=generic doesn't enable sse2, and anvil definitly needs it
>> ---
>> src/intel/vulkan/Makefile.am | 2 +-
>> 1 file changed, 1 insertion(+
Pushed. Thanks!
On Sat, Apr 16, 2016 at 1:48 PM, Marcin Ślusarz
wrote:
>
> ---
> src/intel/vulkan/Makefile.am | 9 +
> src/intel/vulkan/anv_private.h | 3 ---
> 2 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #10 from Timo Aaltonen ---
Oh, I didn't know they were subvariants :)
I've dropped them from our llvm-3.8 for now at least..
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the b
On 2016-04-20 02:20, Nicolai Hähnle wrote:
This is just a slight massaging of the patch you sent previously. What
happened to the discussion we had about how to do this properly?
This already provides good value as-is and it is (IMHO) reasonably
clean, so why not include it for the time being
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Bug ID: 95036
Summary: make check egl-symbols-check regression
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Keywords: r
On Mon, Apr 18, 2016 at 5:10 PM, Jason Ekstrand wrote:
> The first patch in this series adds a short style guide for the Vulkan
> driver. The rest adds a new emit macro and updates the entire driver to
> use it and, while we're there, makes the style more consistent.
>
> Jason Ekstrand (18):
>
On Wed, Apr 20, 2016 at 11:38 AM, Kristian Høgsberg
wrote:
> On Mon, Apr 18, 2016 at 5:10 PM, Jason Ekstrand
> wrote:
> > The first patch in this series adds a short style guide for the Vulkan
> > driver. The rest adds a new emit macro and updates the entire driver to
> > use it and, while we'r
From: Marek Olšák
---
src/gallium/drivers/r600/r600_pipe.c | 3 +
src/gallium/drivers/radeon/Makefile.sources | 1 +
src/gallium/drivers/radeon/r600_pipe_common.c | 2 +
src/gallium/drivers/radeon/r600_pipe_common.h | 5 +
src/gallium/drivers/radeon/r600_test_dma.c| 360 ++
https://bugs.freedesktop.org/show_bug.cgi?id=95038
Bug ID: 95038
Summary: atomic_add and atomic_or cause a OpenCL crashes
Product: Mesa
Version: 11.2
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Se
On 04/20/2016 07:49 PM, Ilia Mirkin wrote:
The SM20/SM30 logic does this for exch || cas. Are you *sure* this
shouldn't be the same? It's pretty silly to do a CAS without a dst,
but it's definitely possible (through DCE).
Yeah, I'm sure. It's definitely not the same logic between gk104 and gk
On Tue, Apr 19, 2016 at 12:44 PM, Matt Turner wrote:
> On Mon, Apr 18, 2016 at 11:52 PM, Kenneth Graunke
> wrote:
>> Previously, opt_vector_float() always interpreted MOV sources as
>> floating point, and always created a MOV with a F-type destination.
>>
>> This meant that we could mess up sequ
On 04/20/2016 07:47 PM, Ilia Mirkin wrote:
Presumably you'd want to touch up the non-limm side of this as well?
Although TBH, I can't really think of a time when it'd matter. I think
you're pretty much guaranteed to have stype == dtype for mul's. Maybe
I'm wrong?
Well, the non-limm part is wr
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Vinson Lee changed:
What|Removed |Added
Keywords||bisected
CC|
On Wed, Apr 20, 2016 at 8:18 PM, Grigori Goronzy wrote:
> On 2016-04-20 02:20, Nicolai Hähnle wrote:
>>
>> This is just a slight massaging of the patch you sent previously. What
>> happened to the discussion we had about how to do this properly?
>>
>
> This already provides good value as-is and it
Float suffixes are allowed in all subsequent GLSL specifications, and
it's obvious what the user meant if they specify one. Accept it with a
warning to avoid breaking applications, like Planeshift.
---
src/compiler/glsl/glsl_lexer.ll | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
On 20.04.2016 11:13, Samuel Pitoiset wrote:
On 04/20/2016 05:43 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
src/mesa/state_tracker/st_cb_queryobj.c | 55
--
I am fine with this, it would be nice if we could modify the following
piglit tests to pass when a warning is emitted:
https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/compiler/literals/invalid-float-suffix-capital-f.vert
https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/co
Am 20.04.2016 um 21:29 schrieb Matt Turner:
> Float suffixes are allowed in all subsequent GLSL specifications, and
> it's obvious what the user meant if they specify one. Accept it with a
> warning to avoid breaking applications, like Planeshift.
> ---
> src/compiler/glsl/glsl_lexer.ll | 4 ++--
>
On Wed, Apr 20, 2016 at 12:48 PM, Lars Hamre wrote:
> I am fine with this, it would be nice if we could modify the following
> piglit tests to pass when a warning is emitted:
> https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/compiler/literals/invalid-float-suffix-capital-f.vert
> htt
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index 1f79fef..ca22df2 100644
--- a/src/egl/opengl/eglinfo.c
+++ b/src/egl/opengl/eglinfo.c
@@ -93,6 +93,8 @@ PrintConfigs(EGLDisplay
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 41 +++--
1 file changed, 35 insertions(+), 6 deletions(-)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index 875e407..75d9fe5 100644
--- a/src/egl/opengl/eglinfo.c
+++ b/src/egl/op
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 29 ++---
1 file changed, 18 insertions(+), 11 deletions(-)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index b044eaa..875e407 100644
--- a/src/egl/opengl/eglinfo.c
+++ b/src/egl/opengl/eglinf
These are independent of the display, and I want to iterate over those.
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index ca22df2..b044eaa 100644
--- a/src/
On Wed, Apr 20, 2016 at 3:14 PM, Samuel Pitoiset
wrote:
>
>
> On 04/20/2016 07:47 PM, Ilia Mirkin wrote:
>>
>> Presumably you'd want to touch up the non-limm side of this as well?
>> Although TBH, I can't really think of a time when it'd matter. I think
>> you're pretty much guaranteed to have sty
On 04/20/2016 09:37 PM, Nicolai Hähnle wrote:
On 20.04.2016 11:13, Samuel Pitoiset wrote:
On 04/20/2016 05:43 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
src/mesa/state_tracker/st_
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 1:47 PM, Samuel Pitoiset
wrote:
> This is only valid for other atomic operations (including CAS). This
> fixes an invalid opcode error from dmesg. While we are it, make sure
> to initialize global addr to 0 for other atomic operations.
>
> Signed-
On Wed, Apr 20, 2016 at 4:10 PM, Adam Jackson wrote:
> Signed-off-by: Adam Jackson
Series is:
Reviewed-by: Alex Deucher
> ---
> src/egl/opengl/eglinfo.c | 41 +++--
> 1 file changed, 35 insertions(+), 6 deletions(-)
>
> diff --git a/src/egl/opengl/eglinfo.c
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 58 ++
1 file changed, 58 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
index 15280df.
1 - 100 of 159 matches
Mail list logo
