https://bugs.freedesktop.org/show_bug.cgi?id=70471
--- Comment #5 from Vinson Lee ---
(In reply to comment #4)
> hm.. do you get the same results using the ./configure and a make?
configure and make passes for me.
--
You are receiving this mail because:
You are the assignee for the bug.
__
https://bugs.freedesktop.org/show_bug.cgi?id=70471
--- Comment #4 from Alexander von Gluck ---
hm.. do you get the same results using the ./configure and a make?
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev ma
https://bugs.freedesktop.org/show_bug.cgi?id=70471
--- Comment #3 from Vinson Lee ---
The build failure occurs with a clean build.
$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=13.04
DISTRIB_CODENAME=raring
DISTRIB_DESCRIPTION="Ubuntu 13.04"
$ llvm-config --cppflags
-I/usr/lib/llvm-
https://bugs.freedesktop.org/show_bug.cgi?id=70471
--- Comment #2 from Alexander von Gluck ---
Just did a clean mesa build on my ArchLinux machine + LLVM 3.3. No issues
seen.
--
You are receiving this mail because:
You are the assignee for the bug.
_
https://bugs.freedesktop.org/show_bug.cgi?id=70471
--- Comment #1 from Alexander von Gluck ---
Odd. This looks like non-rtti code is getting linked against rtti code.
Was this a clean build of mesa? (aka, running scons -c before running scons)
I don't think the build system will pick up on cha
https://bugs.freedesktop.org/show_bug.cgi?id=70471
Priority: medium
Bug ID: 70471
Keywords: regression
CC: curroje...@riseup.net, kallis...@unixzen.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: undefined reference to `typ
Alexander von Gluck IV writes:
> * As discussed on the mailing list,
> forced no-rtti breaks C++ public
> API's such as the Haiku C++ libGL.so
> * -fno-rtti *can* be still set however
> instead of blindly forcing -fno-rtti,
> we can rely on the llvm-config
> --cppflags output.
> If th
Use GL_MAP_INVALIDATE_RANGE, UNSYNCHRONIZED and FLUSH_EXPLICIT flags
when mapping VBOs during display list compilation. This mirrors what
we do for immediate-mode VBO building in vbo_exec_vtx_map().
This improves performance for applications which interleave display
list compilation with executio
On Mon, Oct 14, 2013 at 11:05 AM, Matt Turner wrote:
> On Mon, Oct 14, 2013 at 10:12 AM, Anuj Phogat wrote:
>> diff --git a/src/mapi/glapi/gen/GL3x.xml b/src/mapi/glapi/gen/GL3x.xml
>> index 5078f7b..4ec4749 100644
>> --- a/src/mapi/glapi/gen/GL3x.xml
>> +++ b/src/mapi/glapi/gen/GL3x.xml
>> @@ -6
Kenneth Graunke writes:
> On 10/14/2013 10:44 AM, Eric Anholt wrote:
>> Chris Forbes writes:
>>
>>> Previously this was float, which caused the copy from g0 to mangle
>>> everything.
>>
>> If we face a choice of types for a raw mov, we should choose float --
>> it's higher performance on IVB (
Previously, the best thing we had was to schedule the things unblocked by
the current instruction, on the hope that it would be consuming two values
at the end of their live intervals while only producing one new value.
Sometimes that wasn't the case.
Now, when an instruction is the first user of
bugzilla-dae...@freedesktop.org writes:
> --- Comment #2 from Kristian Høgsberg ---
> Happy I waited to push this, I came up with a much better fix:
>
> commit 360a141f24a9d00891665b7fedb77ffb116944ca
> Author: Kristian Høgsberg
...
> Cc: 9.2
Hi Kristian,
I tried cherry-picking this to mes
On 4 October 2013 15:44, Eric Anholt wrote:
> Now that both vec4 and fs are dynamically assigning offsets, a lot of the
> code is the same.
> ---
>
Since next_binding_table_offset is only used to into
assign_common_binding_table_offsets(), I'd prefer to see it made into a
function argument rathe
On 10/14/2013 12:22 PM, Marek Olšák wrote:
> I'd just add an alias "GL_ARB_texture_mirror_clamp_to_edge" ->
> o(ATI_texture_mirror_once), so it would just be a one-liner change to
> extensions.c.
Like I said:
>> Some hardware
>> with drivers in Mesa support the ATI extension, and some hardware ca
https://bugs.freedesktop.org/show_bug.cgi?id=69437
--- Comment #3 from U. Artie Eoff ---
Thanks, I confirmed it fixes the issue on master... waiting on 9.2 backport.
--
You are receiving this mail because:
You are the QA Contact for the bug.
___
mesa-
On 10/14/2013 10:44 AM, Eric Anholt wrote:
> Chris Forbes writes:
>
>> Previously this was float, which caused the copy from g0 to mangle
>> everything.
>
> If we face a choice of types for a raw mov, we should choose float --
> it's higher performance on IVB (they can get dispatched twice as fa
On 10/14/2013 12:22 PM, Marek Olšák wrote:
> I'd just add an alias "GL_ARB_texture_mirror_clamp_to_edge" ->
> o(ATI_texture_mirror_once), so it would just be a one-liner change to
> extensions.c.
>
> Marek
They're not the same though - the new ARB extension only supports
MIRROR_CLAMP_TO_EDGE, whi
I'd just add an alias "GL_ARB_texture_mirror_clamp_to_edge" ->
o(ATI_texture_mirror_once), so it would just be a one-liner change to
extensions.c.
Marek
On Mon, Oct 14, 2013 at 7:25 PM, Ian Romanick wrote:
> Here's a project for anyone looking to get into Mesa development that
> should be easy f
https://bugs.freedesktop.org/show_bug.cgi?id=70435
--- Comment #3 from U. Artie Eoff ---
(In reply to comment #1)
> It's no surprise that this patch caused a regression. I wish Piglit had
> total and complete coverage of everything :(
>
> Artie, on which hardware did you see the regression? Plea
https://bugs.freedesktop.org/show_bug.cgi?id=70435
--- Comment #2 from U. Artie Eoff ---
Created attachment 87619
--> https://bugs.freedesktop.org/attachment.cgi?id=87619&action=edit
lspci -vvnn
--
You are receiving this mail because:
You are on the CC list for the bug.
__
https://bugs.freedesktop.org/show_bug.cgi?id=70435
Chad Versace changed:
What|Removed |Added
Assignee|i...@freedesktop.org |chad.vers...@linux.intel.co
https://bugs.freedesktop.org/show_bug.cgi?id=70435
Chad Versace changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|chad.vers...@
On Mon, Oct 14, 2013 at 12:43 PM, Chad Versace wrote:
> On 10/14/2013 10:54 AM, Eric Anholt wrote:
>
>> Courtney Goeltzenleuchter writes:
>>
>> Does anyone know of a test that measures frame 0 time? Or texture upload
>>> speed?
>>>
>>> For Smokin' Guns, I tried measuring the overall time, but a
https://bugs.freedesktop.org/show_bug.cgi?id=70435
Chad Versace changed:
What|Removed |Added
Assignee|i...@freedesktop.org |chad.vers...@linux.intel.co
On 10/14/2013 10:54 AM, Eric Anholt wrote:
Courtney Goeltzenleuchter writes:
Does anyone know of a test that measures frame 0 time? Or texture upload
speed?
For Smokin' Guns, I tried measuring the overall time, but an improved frame
0 time has difficulty standing out of a 2607 frame test.
I
On Fri, Oct 11, 2013 at 3:29 PM, John Sheu wrote:
> Hello folks:
>
> About the ownership of dmabuf file descriptors that are passed into EGL.
> I'm looking in particular at this blurb from the spec:
>
>* If is EGL_LINUX_DMA_BUF_EXT and eglCreateImageKHR fails,
> EGL does not reta
We can't perform DCE using the liveness pass between GVN and GCM
because it relies on the correct schedule, but GVN doesn't care about
preserving correctness - it's rescheduled later by GCM.
This patch makes dce_cleanup pass perform simple DCE
between GVN and GCM instead of relying on liveness pas
OK, so this needs to lose the retype() on both sides.
On Tue, Oct 15, 2013 at 6:44 AM, Eric Anholt wrote:
> Chris Forbes writes:
>
>> Previously this was float, which caused the copy from g0 to mangle
>> everything.
>
> If we face a choice of types for a raw mov, we should choose float --
> it's
On Mon, Oct 14, 2013 at 10:12 AM, Anuj Phogat wrote:
> diff --git a/src/mapi/glapi/gen/GL3x.xml b/src/mapi/glapi/gen/GL3x.xml
> index 5078f7b..4ec4749 100644
> --- a/src/mapi/glapi/gen/GL3x.xml
> +++ b/src/mapi/glapi/gen/GL3x.xml
> @@ -630,6 +630,11 @@
>
>
>
> +
> +
> +
> +
> +
>
On Mon, Oct 14, 2013 at 10:12 AM, Anuj Phogat wrote:
> diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
> index 2507fdf..9e908c0 100644
> --- a/src/mesa/main/extensions.c
> +++ b/src/mesa/main/extensions.c
> @@ -200,6 +200,7 @@ static const struct extension extension_table[] =
Courtney Goeltzenleuchter writes:
> Does anyone know of a test that measures frame 0 time? Or texture upload
> speed?
>
> For Smokin' Guns, I tried measuring the overall time, but an improved frame
> 0 time has difficulty standing out of a 2607 frame test.
>
> I may have to create something. Sugg
Courtney Goeltzenleuchter writes:
> Does anyone know of a test that measures frame 0 time? Or texture upload
> speed?
>
> For Smokin' Guns, I tried measuring the overall time, but an improved frame
> 0 time has difficulty standing out of a 2607 frame test.
>
> I may have to create something. Sugg
Abdiel Janulgue writes:
> On Friday, October 11, 2013 11:39:53 AM Eric Anholt wrote:
>> As a general rule, we don't land code whose purpose is performance
>> improvement if it doesn't actually improve performance. If more work is
>> needed to make it actually improve performance, then we wait un
Chris Forbes writes:
> Previously this was float, which caused the copy from g0 to mangle
> everything.
If we face a choice of types for a raw mov, we should choose float --
it's higher performance on IVB (they can get dispatched twice as fast,
when there's a hyperthread waiting to dispatch a fl
Here's a project for anyone looking to get into Mesa development that
should be easy for anyone with decent C programming skills to tackle:
GL_ARB_texture_mirror_clamp_to_edge.
http://www.opengl.org/registry/specs/ARB/texture_mirror_clamp_to_edge.txt
This extension is effectively a subset of anot
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 6f024b4..0d71617 100644
--- a/src/mesa/drivers/dri/i965/intel_extensio
- Enable GEN7_WM_MSDISPMODE_PERSAMPLE, GEN7_WM_POSOFFSET_SAMPLE,
GEN7_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
- Don't enable GEN7_WM_16_DISPATCH_ENABLE when GEN7_WM_MSDISPMODE_PERSAMPLE
is enabled. Refer IVB PRM Vol. 2, Part 1, Page 288 for details.
Signed-off-by: Anuj Phog
- Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
- Don't enable GEN6_WM_16_DISPATCH_ENABLE when GEN6_WM_MSDISPMODE_PERSAMPLE
is enabled. Refer SNB PRM Vol. 2, Part 1, Page 279 for details.
Signed-off-by: Anuj Phog
Implement the FS backend for new builtins added by the extension:
in vec2 gl_SamplePosition
in int gl_SampleID
in int gl_NumSamples
out int gl_SampleMask[]
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 109 +++
src/mesa/drivers/dri/i965/brw
New builtins added by GL_ARB_sample_shading:
in vec2 gl_SamplePosition
in int gl_SampleID
in int gl_NumSamples
out int gl_SampleMask[]
Signed-off-by: Anuj Phogat
---
src/glsl/builtin_variables.cpp | 11 +++
src/glsl/link_varyings.cpp | 2 ++
src/mesa/main/mtypes.h | 7 +
New functions added by GL_ARB_sample_shading:
glMinSampleShadingARB()
New enums:
GL_SAMPLE_SHADING_ARB
GL_MIN_SAMPLE_SHADING_VALUE_ARB
Signed-off-by: Anuj Phogat
---
src/mapi/glapi/gen/ARB_sample_shading.xml | 19 +++
src/mapi/glapi/gen/GL3x.xml | 5 +
src/map
Number of samples will be required in fragment shader program by new
GLSL builtin uniform "gl_NumSamples".
Signed-off-by: Anuj Phogat
---
src/mesa/program/prog_statevars.c | 11 +++
src/mesa/program/prog_statevars.h | 2 ++
2 files changed, 13 insertions(+)
diff --git a/src/mesa/progra
This patch implements the common support code required for the
GL_ARB_sample_shading extension.
Signed-off-by: Anuj Phogat
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
src/glsl/standalone_scaffolding.cpp |
Patches listed below implement the GL_ARB_sample_shading extension
on Intel hardware >= gen6. I verified the implementation with a
number of piglit tests, currently under review on piglit mailing
list. Observed no piglit, gles3 CTS regressions with these patches
on SNB & IVB.
These patches can als
Hi Courtney,
I've been doing similar work but using Xenotic as the benchmark. Here is
how I've been estimating upload times: First I made a rough determination
of a texture count, like 300, as a metric. In intelTexImage I use
clock_gettime to determine the elapsed time between the loading of textur
Instead of checking width==height in four places, just do it in
_mesa_legal_texture_dimensions() where we do the other width, height,
depth checks. Similarly, move the check that cube map array depth is
a multiple of 6.
This change also fixes some missing cube dimension checks for the
glTexStorag
On Friday, October 11, 2013 11:39:53 AM Eric Anholt wrote:
> As a general rule, we don't land code whose purpose is performance
> improvement if it doesn't actually improve performance. If more work is
> needed to make it actually improve performance, then we wait until then.
>
> As I understand
The xmlpool/options.h file was not accessible when building
out-of-tree leading to failure.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70378
Reported-by: Fabio Pedretti
Tested-by: Fabio Pedretti
Tested-by: Andre Heider
Signed-off-by: Emil Velikov
Reviewed-by: Brian Paul
---
src/g
On 10/14/2013 06:40 AM, Mike Lothian wrote:
> Hi
>
> I'm going to try adding in support for texelFetch*Offset to try and get
> anti aliasing working in the Unigine demos
texelFetchOffset is already supported by GLSL 1.30.
There's a lot more to EXT_gpu_shader4 than just these built in
functions.
Does anyone know of a test that measures frame 0 time? Or texture upload
speed?
For Smokin' Guns, I tried measuring the overall time, but an improved frame
0 time has difficulty standing out of a 2607 frame test.
I may have to create something. Suggestions for an appropriate framework?
Thanks,
- Original Message -
> Am 11.10.2013 16:21, schrieb Brian Paul:
> > On 10/11/2013 07:11 AM, srol...@vmware.com wrote:
> >> From: Roland Scheidegger
> >>
> >> The previous limit of of 128*1024 was reported to cause frequent
> >> recompiles
> >> in some apps due to shader variant thrashing
On 10/12/2013 10:29 AM, Emil Velikov wrote:
The xmlpool/options.h file was not accessible when building
out-of-tree leading to failure.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70378
Reported-by: Fabio Pedretti
Tested-by: Andre Heider
Signed-off-by: Emil Velikov
---
src/galliu
https://bugs.freedesktop.org/show_bug.cgi?id=70411
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=70435
U. Artie Eoff changed:
What|Removed |Added
Summary|glTexSubImage corrupted |[regression] Fast Texture
On 10/13/2013 08:33 PM, Ian Romanick wrote:
On 10/13/2013 01:50 PM, Frank Henigman wrote:
On Fri, Oct 11, 2013 at 10:00 PM, Chad Versace
wrote:
On 10/11/2013 10:17 AM, Courtney Goeltzenleuchter wrote:
Support all levels of a supported texture format.
---
src/mesa/drivers/dri/i965/intel_te
Hi
I'm going to try adding in support for texelFetch*Offset to try and get
anti aliasing working in the Unigine demos
I'm not sure if anything more will be required but I just wanted to check
no one else is working on this
Thanks
Mike
___
mesa-dev mai
https://bugs.freedesktop.org/show_bug.cgi?id=70378
--- Comment #10 from Kai ---
(In reply to comment #7)
> Created attachment 87497 [details] [review]
> swrast: add correct include for out-of-tree builds
>
> A slightly more overzealous version, explicitly including the builddir for
> classic swr
https://bugs.freedesktop.org/show_bug.cgi?id=70123
--- Comment #11 from Christian König ---
(In reply to comment #10)
> The command which causes the freeze is :-
>
> compton -b --backend glx --config /dev/null
Thanks for this, I can reproduce the problem now.
Not sure if that's an issue in com
https://bugs.freedesktop.org/show_bug.cgi?id=70123
--- Comment #10 from Jeff Blake ---
Created attachment 87588
--> https://bugs.freedesktop.org/attachment.cgi?id=87588&action=edit
thread apply all bt
With that patch things are still freezing up, see the attachment for the gdb
output (which st
Just let it be handled by the lowering pass.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 56 ++
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 25 ++--
2 files changed, 16 insertions(+), 65 deletions(-)
diff --git a/src/mesa
ir_txf expects an ivec* coordinate, and may be larger than ivec2;
shuffle things around so that this will work.
Signed-off-by: Chris Forbes
---
.../dri/i965/brw_lower_unnormalized_offset.cpp | 51 ++
1 file changed, 42 insertions(+), 9 deletions(-)
diff --git a/src/mesa/
It turns out that nonzero offsets with gsampler2DRect don't work -- they
just return garbage. Work around this by folding the offset into the
coord.
Done as an IR pass rather than yet another hack in the visitors because
it's clear what's going on this way. Can possibly reuse this to replace
the e
This is the fifth (and final) piece of ARB_gpu_shader5 textureGather* support.
It turns out that unnormalized texcoords and texture offsets don't mix.
This series adds yet another lowering pass, so that *sampler2DRect work
with gather.
2-3/3 expand the lowering pass to cover integer coordinates t
Rewrites textureGatherOffsets(s, p, offsets) into
gvec4(
textureGatherOffset(s, p, offsets[0]).w,
textureGatherOffset(s, p, offsets[1]).w,
textureGatherOffset(s, p, offsets[2]).w,
textureGatherOffset(s, p, offsets[3]).w
)
Signed-off-by: Chris Forbes
---
src/mesa
We don't have a message that does 4 independent offsets; a lowering
pass needs to lower it to 4 normal gather4s before reaching this
point.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 3 +++
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 3 +++
2 files chan
Signed-off-by: Chris Forbes
---
src/glsl/builtin_functions.cpp | 30 ++
1 file changed, 30 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index 1b23677..45fff4c 100644
--- a/src/glsl/builtin_functions.cpp
+++ b/src/glsl/buil
This is needed for textureGatherOffsets()
Signed-off-by: Chris Forbes
---
src/glsl/builtin_functions.cpp | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index deedddb..1b23677 100644
--- a/src/glsl/builtin_functions.cpp
Adds support for textureGatherOffsets() [which takes an array of texel offsets].
This isn't directly supported on i965, so we lower it to 4x
textureGatherOffset().
V2: - Rebase, etc.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists
Note that gather4_po_c's parameters are too long for SIMD16. It might be
worth emitting 2xSIMD8 messages in this case at some point.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 15 ---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 3 +++
2 fil
gather4_c's argument layout is straightforward -- refz just goes on the
end.
gather4_po_c's layout however -- the array index is replaced with refz.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 12 ++--
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
Signed-off-by: Chris Forbes
Reviewed-by: Eric Anholt
Reviewed-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_defines.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
b/src/mesa/drivers/dri/i965/brw_defines.h
index f1ea736..c0caba6 100644
--
Signed-off-by: Chris Forbes
Reviewed-by: Eric Anholt
Reviewed-by: Kenneth Graunke
---
src/glsl/builtin_functions.cpp | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index ef8b7bb..deedddb 100644
--- a/src/glsl/built
ARB_gpu_shader5's textureGather*() functions which take shadow samplers
have a separate `refz` parameter rather than adding it to the
coordinate.
Signed-off-by: Chris Forbes
Reviewed-by: Eric Anholt
Reviewed-by: Kenneth Graunke
---
src/glsl/builtin_functions.cpp | 20 +++-
1 fi
Adds infrastructure for separate reference Z in texturing functions,
and support for shadow comparitors with textureGather*.
V2: - General churn, send-from-grf rebase, etc
- Make it actually work (Thanks Eric for pointing out that it didnt)
___
mesa
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index bd9c9e9..d6c565a 100644
--- a/
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 46 +---
1 file changed, 42 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
index 3fde443..fe4741d 100644
Some texturing ops are about to have nonconstant offset support; the
offset in the header in these cases should be zero.
Signed-off-by: Chris Forbes
Reviewed-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_shader.cpp | 9 +++--
The generator code ends up clearer this way than if we had to sniff
via the message length. Implemented via the gather4_po message in
hardware, which is present in Gen7 and later.
Signed-off-by: Chris Forbes
Reviewed-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_defines.h | 1 +
s
Signed-off-by: Chris Forbes
Reviewed-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965/brw_shader.cpp
index 61c4bf5..19500d1 100644
--- a/src/mesa/drivers/dri/i
Prior to ARB_gpu_shader5 / GLSL 4.0, the offset is required to be
a constant expression.
With that extension, it is relaxed to be an arbitrary expression.
Signed-off-by: Chris Forbes
Reviewed-by: Ian Romanick
---
src/glsl/builtin_functions.cpp | 61 --
1
- gsampler2DRect
- optional `comp` parameter
Signed-off-by: Chris Forbes
Reviewed-by: Ian Romanick
---
src/glsl/builtin_functions.cpp | 16
1 file changed, 16 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index d40888d..aa40876 1006
This series adds new textureGatherOffset overloads, and support for
nonconstant offsets.
V2: - Update for general churn + send-from-grf
- Reinstate assert on nonconstant offset if ARB_gpu_shader5 not supported.
This ought not to actually get hit as the GLSL frontend insists on a
co
On Friday, October 11, 2013 11:39:53 AM Eric Anholt wrote:
>
> As I understand it, the thing that you think will make this eventually
> actually improve performance is state flagging that indicates which
> individual surfaces need updating. Since that should improve
> performance even in the non
83 matches
Mail list logo
