This patch changes the error reporting behavior for incorrect
function invocation (triggered by match_function_by_name()
unable to find a matching function call) from using the line
number information associated to the function name term
to using the line number information of the entire function
Enhancement: if no function signature is found for a function
name, report that the function is not found instead of printing
an empty list of candidates.
---
src/glsl/ast_function.cpp | 27 +--
1 file changed, 17 insertions(+), 10 deletions(-)
diff --git
Simple memory allocation checks became long patches.
Juha-Pekka Heikkila (3):
Mesa: Change save_attrib_data() to return boolean
mesa: Verify memory allocations success in _mesa_PushClientAttrib
mesa: Verify memory allocations success in _mesa_PushAttrib
src/mesa/main/attrib.c | 394
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/main/attrib.c | 320 +
1 file changed, 272 insertions(+), 48 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 87797d1..3083e21 100644
---
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/main/attrib.c | 70 ++
1 file changed, 59 insertions(+), 11 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 3ecf533..87797d1 100644
---
Change save_attrib_data() to return true/false depending on success.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/main/attrib.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/dri2.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glx/dri2.c b/src/glx/dri2.c
index bcd1f9c..deb50ab 100644
--- a/src/glx/dri2.c
+++ b/src/glx/dri2.c
@@ -102,6 +102,8 @@ DRI2WireToEvent(Display *dpy, XEvent
Hi,
I made this patch a while ago while adding support for eglSwapInterval
to the Wayland platform. The swap_interval member confused me for a
little while before I realised that it's not used anywhere. I think it
would be good to get rid of it.
Regards,
- Neil
--- 8 --- (use
On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
On evergreen we have to reserve 1 stack element in some additional cases
besides the ones mentioned in the docs, but stack size computation was
recently reimplemented exactly as described in the docs by the patch that
added
GEN6-specific. Using the level and lod fields allows us to resolve directly
to mipmap's level. Remove legacy code to specify the level using the
tile_x and y offsets.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/gen6_blorp.cpp | 25
We don't need this and the hardware allows us to sample directly from
a mipmap's specific level and layer. We can simplify
the operation by adding a level field in DRIImage instead.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/intel_regions.h |
Remove legacy code when recreating a single-layer miptree from a level
of the source texture using tile_x and tile_y offsets. Save the level field
in intel_texture_object so we can resolve this direcly in the hardware.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
Hi,
The first two patches register the R600EmitClauseMarkers and
AMDGPUCFGStructurizer passes, so that -print-*-all will now work with
them. The third patch is a cleanup of Processors.td and the fourth
patch adds the wavefront size property to the subtargets.
Please Review.
-Tom
From
GEN7+-specific hw-support. This can be copied over to GEN8/Broadwell with few
modifications.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/gen7_blorp.cpp | 24
1 file changed, 8 insertions(+), 16 deletions(-)
diff --git
On 12/07/2013 11:17 PM, Chris Forbes wrote:
The target parameter to _mesa_get_tex_image() is a target enum, not an index.
When we're setting up faces for a cubemap, it should be
CUBE_MAP_POSITIVE_X .. CUBE_MAP_NEGATIVE_Z; for all other targets it
should be the same as the texobj's target.
Fixes
(cover letter seems to be lost in the way for the patch series but here it is)
Gen 6/7 hardware support for tile_x and tile_y offset fields in SURFACE_STATE
has been removed in Gen8. All code dependencies must be removed. This patch
series completely eliminates the tile_x and y fields in the
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
And to check if it can have layers at all. This will be used by the
implementation of ARB_shader_image_load_store.
---
src/mesa/main/teximage.c | 85
src/mesa/main/teximage.h | 6
2 files
On 12/09/2013 06:15 AM, Juha-Pekka Heikkila wrote:
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/dri2.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glx/dri2.c b/src/glx/dri2.c
index bcd1f9c..deb50ab 100644
--- a/src/glx/dri2.c
+++ b/src/glx/dri2.c
On 12/09/2013 08:19 AM, Pohjolainen, Topi wrote:
On Mon, Dec 09, 2013 at 02:29:21PM +0200, Juha-Pekka Heikkila wrote:
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/main/attrib.c | 320 +
1 file changed, 272
https://bugs.freedesktop.org/show_bug.cgi?id=69285
--- Comment #3 from Charles Huber genpfa...@gmail.com ---
Still broken in 10.0.
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
On Mon, Dec 9, 2013 at 8:18 AM, Abdiel Janulgue
abdiel.janul...@linux.intel.com wrote:
(cover letter seems to be lost in the way for the patch series but here it is)
Gen 6/7 hardware support for tile_x and tile_y offset fields in SURFACE_STATE
has been removed in Gen8. All code dependencies
On 12/07/2013 10:02 PM, Kristian Høgsberg wrote:
The driverPrivate pointer is opaque to the driver and we can't assume
it's a struct gl_context in dri_util.c. Instead provide a helper function
to set the struct gl_context flags from the incoming DRI context flags.
Signed-off-by: Kristian
Jordan, Abdiel:
Could you provide some context -- the impression I get is that using
the tile offsets is undesirable for reasons other than sharing code
with Gen8, but not familiar enough with the details to know why.
Just curious :)
-- Chris
On Tue, Dec 10, 2013 at 7:21 AM, Jordan Justen
Thanks Paul, that makes a lot of sense.
-- Chris
On Tue, Dec 10, 2013 at 8:34 AM, Paul Berry stereotype...@gmail.com wrote:
On 9 December 2013 11:06, Chris Forbes chr...@ijw.co.nz wrote:
Jordan, Abdiel:
Could you provide some context -- the impression I get is that using
the tile offsets
On 12/09/2013 10:53 AM, Ian Romanick wrote:
On 12/07/2013 10:02 PM, Kristian Høgsberg wrote:
The driverPrivate pointer is opaque to the driver and we can't assume
it's a struct gl_context in dri_util.c. Instead provide a helper function
to set the struct gl_context flags from the incoming DRI
On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
On evergreen we have to reserve 1 stack element in some additional cases
besides the ones mentioned in the docs, but stack size computation was
recently reimplemented exactly as described in the docs by the patch that
added
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/03/2013 10:44 AM, Eric Anholt wrote:
Siavash Eliasi siavashser...@gmail.com writes:
Hello, this is V2 series of patches to accomplish *Enable
ARB_map_buffer_alignment in all drivers* newbie project suggested
by Ian Romanick.
I think
On 5 December 2013 11:33, Carl Worth cwo...@cworth.org wrote:
Carl Worth cwo...@cworth.org writes:
So, you should all expect to see these hook messages now:
remote: I: patch #PATCHWORK_ID updated using rev GIT_COMMIT.
...
remote: E: failed to find patch for rev GIT_COMMIT.
To help the transition period when DRI loaders are being updated
to support the newer __driDriverExtensions_foo mechanism,
we populate __driDriverExtensions with the extensions returned
by __driDriverExtensions_foo during a library contructor
function.
We find the driver foo's name by using the
On Mon, Dec 9, 2013 at 12:41 PM, Ian Romanick i...@freedesktop.org wrote:
On 12/09/2013 10:53 AM, Ian Romanick wrote:
On 12/07/2013 10:02 PM, Kristian Høgsberg wrote:
The driverPrivate pointer is opaque to the driver and we can't assume
it's a struct gl_context in dri_util.c. Instead provide
On Mon, 2013-12-09 at 10:56 -0500, Tom Stellard wrote:
On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
On evergreen we have to reserve 1 stack element in some additional cases
besides the ones mentioned in the docs, but stack size computation was
recently reimplemented exactly
On 7 December 2013 07:42, Francisco Jerez curroje...@riseup.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 24 November 2013 21:00, Francisco Jerez curroje...@riseup.net
wrote:
Including pack/unpack and texstore code. This texture format is a
requirement for
On 7 December 2013 17:11, Francisco Jerez curroje...@riseup.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 24 November 2013 21:00, Francisco Jerez curroje...@riseup.net
wrote:
[...]
+
+ case GL_RGBA16UI:
+ return MESA_FORMAT_RGBA_UINT16;
+
+ case GL_RGB10_A2UI:
On 7 December 2013 17:12, Francisco Jerez curroje...@riseup.net wrote:
Paul Berry stereotype...@gmail.com writes:
With those changes, this patch is:
Reviewed-by: Paul Berry stereotype...@gmail.com
I think I've taken into account all your comments here [1], and I've
changed the value
On 7 December 2013 17:17, Francisco Jerez curroje...@riseup.net wrote:
And to check if it can have layers at all. This will be used by the
implementation of ARB_shader_image_load_store.
---
src/mesa/main/teximage.c | 85
Calling the local variables flat_enable and point_sprite_enable is
clearer than dw16 and such. It also matches the names used in
calculate_attr_overrides, which computes them.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
Cc: Paul Berry stereotype...@gmail.com
---
calculate_attr_overrides is responsible for computing the point sprite
and flat-shading enable bitfields. It does so by OR'ing in a bunch of
bits. However, it relied on the caller to set the initial value to
zero. This is pretty fragile - if the caller neglects to zero out those
variables, then
Jordan Justen jordan.l.jus...@intel.com writes:
To help the transition period when DRI loaders are being updated
to support the newer __driDriverExtensions_foo mechanism,
we populate __driDriverExtensions with the extensions returned
by __driDriverExtensions_foo during a library contructor
From: Kristian Høgsberg k...@bitplanet.net
The driverPrivate pointer is opaque to the driver and we can't assume
it's a struct gl_context in dri_util.c. Instead provide a helper function
to set the struct gl_context flags from the incoming DRI context flags.
v2 (idr): Modify the other classic
Ian Romanick i...@freedesktop.org writes:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/03/2013 10:44 AM, Eric Anholt wrote:
Siavash Eliasi siavashser...@gmail.com writes:
Hello, this is V2 series of patches to accomplish *Enable
ARB_map_buffer_alignment in all drivers* newbie
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/09/2013 04:58 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1
On 12/03/2013 10:44 AM, Eric Anholt wrote:
Siavash Eliasi siavashser...@gmail.com writes:
Hello, this is V2
https://bugs.freedesktop.org/show_bug.cgi?id=70250
--- Comment #4 from Ian Romanick i...@freedesktop.org ---
Artie,
Did this fix the problem for you? There's no Reviewed-by on the patch, so I
want *something* before I cherry pick it over to stable.
Thanks.
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=70250
--- Comment #5 from Ian Romanick i...@freedesktop.org ---
Since you marked the bug as VERIFIED, I'll call that a Tested-by. Sorry for
the noise.
--
You are receiving this mail because:
You are the QA Contact for the bug.
OpenGL with ARB_ES2_compatibility extension allows shaders that specify
#version 100 and the shader is treated as targeting version 100 of the
GLSL ES.
state-es_shader flag might change during parsing based on the #version
directive mentioned in the shader program. This patch adds another check
The fact that we flush denorms to zero breaks our half-float
conversion and blending. This patches enables denorms for
blending. It's a little tricky due to the llvm bug that makes
it incorrectly reorder the mxcsr intrinsics:
http://llvm.org/bugs/show_bug.cgi?id=6393
Signed-off-by: Zack Rusin
On Tue, Dec 10, 2013 at 02:54:46AM +0400, Vadim Girlin wrote:
On Mon, 2013-12-09 at 10:56 -0500, Tom Stellard wrote:
On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
On evergreen we have to reserve 1 stack element in some additional cases
besides the ones mentioned in the
Disabled by default, but it's very useful when needed.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/drivers/llvmpipe/lp_setup_point.c | 20
1 file changed, 20 insertions(+)
diff --git a/src/gallium/drivers/llvmpipe/lp_setup_point.c
Caching in the vbuf module meant that once a vertex has been
emitted it was cached, but it's possible for a vertex at the
same location to be emitted again, but this time with a different
front-face semantic. Caching was causing the first version of the
vertex to be emitted, which resulted in the
Ian Romanick i...@freedesktop.org writes:
From: Kristian Høgsberg k...@bitplanet.net
The driverPrivate pointer is opaque to the driver and we can't assume
it's a struct gl_context in dri_util.c. Instead provide a helper function
to set the struct gl_context flags from the incoming DRI
49 matches
Mail list logo