From: Petri Latvala
Sort expression operands when possible so that building expected IR
sexps doesn't need to know which ordering will be produced by an
optimization pass.
Signed-off-by: Petri Latvala
Version 2 by Iago Toral Quiroga :
In python style constants go at the top of the file, and s
From: Petri Latvala
Signed-off-by: Petri Latvala
---
src/glsl/tests/test_case_generator.py | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/glsl/tests/test_case_generator.py
b/src/glsl/tests/test_case_generator.py
index 6633702..5f550df 100644
--- a/src/glsl/tests/test_
From: Petri Latvala
Signed-off-by: Petri Latvala
---
src/glsl/test_optpass.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glsl/test_optpass.cpp b/src/glsl/test_optpass.cpp
index 24c06f1..d88a904 100644
--- a/src/glsl/test_optpass.cpp
+++ b/src/glsl/test_optpass.cpp
@@ -131,6 +131
Mostly blank lines and comment formatting to ease readability as suggested by
Dylan Baker.
---
src/glsl/tests/test_case_generator.py | 86 +++
1 file changed, 66 insertions(+), 20 deletions(-)
diff --git a/src/glsl/tests/test_case_generator.py
b/src/glsl/tests/tes
From: Petri Latvala
optimization-test loops over directories that contain a file named
create_test_cases.py. For each occurrence, it enters the directory
that contains the file and executes it, and is then supposed to return
to the previous directory.
The name .. was hardcoded in the script, and
This addresses review feedback for the original series sent by Petri:
http://lists.freedesktop.org/archives/mesa-dev/2014-July/064222.html
I have already sent a new version of the first patch in that series that
implements the optimization pass, and has already been reviewed. This series
addresses
Matt Turner writes:
> Reviewed-by: Matt Turner
>
> Should we add a Cc: for the stable branch?
Also seems like a good plan. I've added that and pushed.
--
keith.pack...@intel.com
pgp3KeIMuqnda.pgp
Description: PGP signature
___
mesa-dev mailing lis
On 01.10.2014 01:46, Marek Olšák wrote:
From: Marek Olšák
The radeonsi patches in this series are
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer| http://www.amd.com
Libre software enthusiast |Mesa and X developer
On Tue, Sep 30, 2014 at 5:49 PM, Jason Ekstrand wrote:
> Signed-off-by: Jason Ekstrand
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84529
> ---
> src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965
Instead of just segfaulting in the driver when a buffer allocation fails,
report error messages indicating what went wrong so that we can debug things.
As a simple example, chromium wraps Mesa in a sandbox which doesn't allow
access to most syscalls, including the ability to create shared memory
s
https://bugs.freedesktop.org/show_bug.cgi?id=84538
Vinson Lee changed:
What|Removed |Added
CC||jfons...@vmware.com,
|
On Tue, Sep 30, 2014 at 5:26 PM, Matt Turner wrote:
> On Tue, Sep 30, 2014 at 5:18 PM, Jason Ekstrand
> wrote:
> > Reviewed-by: Jason Ekstrand
>
> The patch is wrong. See my reply.
>
Hunh? Reply to what?
___
mesa-dev mailing list
mesa-dev@lists.freed
Signed-off-by: Jason Ekstrand
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84529
---
src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_fp.cpp
b/src/mesa/drivers/dri/i965/brw_fs_fp.cpp
index f9eec
On Tue, Sep 30, 2014 at 6:57 PM, Roland Scheidegger wrote:
>
>> +/* Inserts a MOV_SAT for the needed components of tex coord. Note that
>> + * in the case of TXP, the clamping must happen *after* projection, so
>> + * we need to lower TXP to TEX.
>> + *
>> + * MOV tmpA, src0
>> + * if (opc ==
On Tue, Sep 30, 2014 at 6:31 PM, Eric Anholt wrote:
> Brian Paul writes:
>
>> On 09/30/2014 03:38 PM, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> TGSI->TGSI pass, extracted from freedreno. Currently provides the
>>> following lower support, to help drivers emulate unsupported opcodes
>>> or fe
On 09/30/2014 03:04 PM, Matt Turner wrote:
> On Tue, Sep 30, 2014 at 2:26 PM, Ian Romanick wrote:
>> Are there a bunch of cases where we double emit things for SIMD16?
>> Would it make more sense to have a generic function that took a list of
>> instructions, duplicated them, and did the force_unc
On 09/30/2014 04:24 PM, Matt Turner wrote:
> ---
> src/util/macros.h | 13 +
> 1 file changed, 13 insertions(+)
>
> diff --git a/src/util/macros.h b/src/util/macros.h
> index 40ebf02..77055e8 100644
> --- a/src/util/macros.h
> +++ b/src/util/macros.h
> @@ -75,6 +75,19 @@ do {
On Tue, Sep 30, 2014 at 3:42 PM, Jason Ekstrand wrote:
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 12 +++-
> 1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
> b/src/mesa/drivers/dri/i965/brw_fs.cpp
> index 6ed3e95..7e4cb04 100644
From: Dave Airlie
While running piglit in virgl, I hit an assert in intel driver.
"qemu-system-x86_64: intel_tex.c:219: intel_map_texture_image: Assertion
`tex_image->TexObject->Target != 0x8C18 || h == 1' failed."
Thanks to Eric and Ken for pointing me in the right direction,
Fix the get_tex
The compiler isn't privy to the knowledge that we're doing at least one
framebuffer write.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 19e9cb9..d113812 100644
--- a/s
---
src/util/macros.h | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/util/macros.h b/src/util/macros.h
index 40ebf02..77055e8 100644
--- a/src/util/macros.h
+++ b/src/util/macros.h
@@ -75,6 +75,19 @@ do {\
#define unreachable(str)
#endif
+/**
+ *
Am 30.09.2014 23:38, schrieb Rob Clark:
> From: Rob Clark
>
> TGSI->TGSI pass, extracted from freedreno. Currently provides the
> following lower support, to help drivers emulate unsupported opcodes
> or features:
>
> Individual opcodes:
> DST, XPD, SCS, LRP, FRC, POW, LIT, EXP, LOG, DP4, DP3
On 09/30/2014 04:31 PM, Eric Anholt wrote:
Brian Paul writes:
On 09/30/2014 03:38 PM, Rob Clark wrote:
From: Rob Clark
TGSI->TGSI pass, extracted from freedreno. Currently provides the
following lower support, to help drivers emulate unsupported opcodes
or features:
Individual opcodes:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 6ed3e95..7e4cb04 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri
Series
Reviewed-by: Jordan Justen
On 2014-09-30 11:40:55, Anuj Phogat wrote:
> SampleMap{2,4,8}x variables are used in later patches to implement
> EXT_framebuffer_multisample_blit_scaled extension.
>
> V2: Use integer array instead of a string.
> Bump up the comment.
>
> V3: Use uint8_t ty
On Tuesday, September 30, 2014 11:37:37 AM Matt Turner wrote:
> On Tue, Sep 30, 2014 at 5:22 AM, Samuel Iglesias Gonsálvez
> wrote:
> > Hello,
> >
> > I am looking at bug 57439 [0] where it shows an error
> > in a piglit test [1] related to textureSize() function happening
> > in Intel SandyBridge
Hi all,
Thanks and pushed.
Good Night!
Mathias
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
---
src/gallium/auxiliary/tgsi/tgsi_lowering.c | 2306 ++--
src/gallium/auxiliary/tgsi/tgsi_lowering.h | 99 +-
2 files changed, 1201 insertions(+), 1204 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_lowering.c
b/src/gallium/auxiliary/tgsi/tgsi_lowering.c
in
---
src/gallium/auxiliary/Makefile.sources | 1 +
src/gallium/auxiliary/tgsi/tgsi_lowering.c | 50 +++---
src/gallium/auxiliary/tgsi/tgsi_lowering.h | 12 +++
3 files changed, 32 insertions(+), 31 deletions(-)
diff --git a/src/gallium/auxiliary/Makefile.sources
b
Lots of drivers need to transform the weird instructions in TGSI into
reasonable scalar ops, and this code can make those translations
canonical.
---
src/gallium/auxiliary/tgsi/tgsi_lowering.c | 1573
src/gallium/auxiliary/tgsi/tgsi_lowering.h | 89 ++
2 files change
Brian Paul writes:
> On 09/30/2014 03:38 PM, Rob Clark wrote:
>> From: Rob Clark
>>
>> TGSI->TGSI pass, extracted from freedreno. Currently provides the
>> following lower support, to help drivers emulate unsupported opcodes
>> or features:
>>
>> Individual opcodes:
>>DST, XPD, SCS, LRP, FR
Am 01.10.2014 00:17, schrieb Mathias Fröhlich:
>
> Brian,
>
> at least here, I get a build that runs glxgears which
> previously did not run with 3.3, 3.5.
> Currently the compile test runs with 3.6.
> If this succeeds, ok to push the attached fix
> (The same than before but with a more descripti
On 09/30/2014 04:17 PM, Mathias Fröhlich wrote:
Brian,
at least here, I get a build that runs glxgears which
previously did not run with 3.3, 3.5.
Currently the compile test runs with 3.6.
If this succeeds, ok to push the attached fix
(The same than before but with a more descriptive commit mes
Brian,
at least here, I get a build that runs glxgears which
previously did not run with 3.3, 3.5.
Currently the compile test runs with 3.6.
If this succeeds, ok to push the attached fix
(The same than before but with a more descriptive commit message)?
Greetings
Mathias>From 39a8625423f85327eef
Rob Clark writes:
> From: Rob Clark
>
> TGSI->TGSI pass, extracted from freedreno. Currently provides the
> following lower support, to help drivers emulate unsupported opcodes
> or features:
>
> Individual opcodes:
> DST, XPD, SCS, LRP, FRC, POW, LIT, EXP, LOG, DP4, DP3, DPH,
> DP2, DP2A
>
On Tue, Sep 30, 2014 at 2:26 PM, Ian Romanick wrote:
> Are there a bunch of cases where we double emit things for SIMD16?
> Would it make more sense to have a generic function that took a list of
> instructions, duplicated them, and did the force_uncompressed /
> force_sechalf modification?
Not m
On 09/30/2014 03:38 PM, Rob Clark wrote:
From: Rob Clark
TGSI->TGSI pass, extracted from freedreno. Currently provides the
following lower support, to help drivers emulate unsupported opcodes
or features:
Individual opcodes:
DST, XPD, SCS, LRP, FRC, POW, LIT, EXP, LOG, DP4, DP3, DPH,
DP
Hi Brian,
On Tuesday, September 30, 2014 15:42:21 Brian Paul wrote:
> Yeah, reverting that patch clears up the regression here. Please go
> ahead and do the revert if you don't think you can solve the problem
> otherwise. Thanks!
I could even reproduce the segfault with the previous patch an
On 09/30/2014 03:34 PM, Mathias Fröhlich wrote:
Hi,
On Tuesday, September 30, 2014 21:25:54 Jose Fonseca wrote:
Your patch looks good AFAICT.
Not sure why the crash, and I'm afraid I won't have time to look into it.
I am currently looking into that.
I think it might help to '#define USE_MC
From: Rob Clark
TGSI->TGSI pass, extracted from freedreno. Currently provides the
following lower support, to help drivers emulate unsupported opcodes
or features:
Individual opcodes:
DST, XPD, SCS, LRP, FRC, POW, LIT, EXP, LOG, DP4, DP3, DPH,
DP2, DP2A
Also supported, although it is up to
Hi,
On Tuesday, September 30, 2014 21:25:54 Jose Fonseca wrote:
> Your patch looks good AFAICT.
>
> Not sure why the crash, and I'm afraid I won't have time to look into it.
I am currently looking into that.
> I think it might help to '#define USE_MCJIT 1' for now, ie, enable MCJIT for
> all L
On 09/25/2014 09:00 AM, Matt Turner wrote:
> On Thu, Sep 25, 2014 at 8:25 AM, Ian Romanick wrote:
>> How did you test this? Do we have piglit execution tests that actually
>> hit this path? I'm assuming you didn't play Serious Sam 3 looking for
>> rendering errors. ;)
>
> I wrote the patch and
On 09/30/2014 02:16 PM, Matt Turner wrote:
> On Tue, Sep 30, 2014 at 2:10 PM, Ian Romanick wrote:
>> On 09/27/2014 12:12 PM, Matt Turner wrote:
>>> The next patch adds an algebraic optimization for the pattern
>>>
>>>sqrt a, b
>>>rcp c, a
>>>
>>> and turns it into
>>>
>>>sqrt a, b
>>>
The first 3 are
Reviewed-by: Ian Romanick
I sent a question on patch 4 that may affect it and the others.
On 09/28/2014 01:26 PM, Matt Turner wrote:
> [PATCH 1/6] i965/fs: Set MUL source type to W/UW in 64-bit mul macro
>
>Fixes 64-bit multiploes on Gen8.
>
> [PATCH 2/6] i965/fs: Don't of
On 09/28/2014 01:26 PM, Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 22 --
> 1 file changed, 16 insertions(+), 6 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> index e
Brian,
Your patch looks good AFAICT.
Not sure why the crash, and I'm afraid I won't have time to look into it.
I think it might help to '#define USE_MCJIT 1' for now, ie, enable MCJIT for
all LLVM versions . We were avoiding it on old LLVM versions, but AFAICT
there's no longer any reason to
On Mon, Sep 8, 2014 at 12:21 PM, Matt Turner wrote:
> Helps a handful of programs in Serious Sam 3 that use do-while loops.
>
> instructions in affected programs: 16114 -> 16075 (-0.24%)
> ---
How about a review?
___
mesa-dev mailing list
mesa-dev@l
On Tue, Sep 30, 2014 at 2:10 PM, Ian Romanick wrote:
> On 09/27/2014 12:12 PM, Matt Turner wrote:
>> The next patch adds an algebraic optimization for the pattern
>>
>>sqrt a, b
>>rcp c, a
>>
>> and turns it into
>>
>>sqrt a, b
>>rsq c, b
>>
>> but many vertex shaders do
>>
>>
On 09/27/2014 12:12 PM, Matt Turner wrote:
> The next patch adds an algebraic optimization for the pattern
>
>sqrt a, b
>rcp c, a
>
> and turns it into
>
>sqrt a, b
>rsq c, b
>
> but many vertex shaders do
>
>a = sqrt(b);
>var1 /= a;
>var2 /= a;
>
> which generat
On Sat, Sep 27, 2014 at 12:12 PM, Matt Turner wrote:
> The next patch adds an algebraic optimization for the pattern
>
>sqrt a, b
>rcp c, a
>
> and turns it into
>
>sqrt a, b
>rsq c, b
>
> but many vertex shaders do
>
>a = sqrt(b);
>var1 /= a;
>var2 /= a;
>
> which ge
On 09/30/2014 11:24 AM, Matt Turner wrote:
> On Tue, Sep 30, 2014 at 8:50 AM, Ian Romanick wrote:
>> On 09/20/2014 07:41 PM, Matt Turner wrote:
>>> On Sat, Sep 20, 2014 at 6:40 PM, Chris Forbes wrote:
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index 79d2e94..c11a
On 09/30/2014 02:40 PM, Mathias Fröhlich wrote:
Hi Brian,
On Tuesday, September 30, 2014 13:30:21 Brian Paul wrote:
Move the USE_MCJIT / HAVE_AVX determination logic to lp_bld.h. If we
don't have MCJIT define a dummy LLVMMCJITMemoryManagerRef type to avoid
excessive #ifdef testing elsewhere.
Series looks good. Thanks for looking into this Roland.
It looks nobody else is using galahad, nor looking at the warnings. I wonder
if it makes sense to keep using/updating it.
Jose
From: srol...@vmware.com
Sent: 30 September 2014 19:07
To: Jose Fonse
Hi Brian,
On Tuesday, September 30, 2014 13:30:21 Brian Paul wrote:
> Move the USE_MCJIT / HAVE_AVX determination logic to lp_bld.h. If we
> don't have MCJIT define a dummy LLVMMCJITMemoryManagerRef type to avoid
> excessive #ifdef testing elsewhere.
[...]
> @@ -219,7 +188,7 @@ gallivm_free_code
On Tue, Sep 30, 2014 at 9:04 PM, Roland Scheidegger wrote:
> Am 30.09.2014 18:46, schrieb Marek Olšák:
>> From: Marek Olšák
>>
>> Use an array of properties indexed by TGSI_PROPERTY_* definitions.
>> ---
>> src/gallium/auxiliary/draw/draw_gs.c | 23 -
>> src/gallium/auxiliary
On Tue, Sep 30, 2014 at 1:15 AM, Kenneth Graunke wrote:
> Write-back caching cannot be used for buffers being scanned out by the
> display engine; surfaces used for scan-out must be write-through or
> uncached. I originally chose WT for render targets because it works in
> all cases. However, we
Move the USE_MCJIT / HAVE_AVX determination logic to lp_bld.h. If we
don't have MCJIT define a dummy LLVMMCJITMemoryManagerRef type to avoid
excessive #ifdef testing elsewhere.
---
src/gallium/auxiliary/gallivm/lp_bld.h | 40 +++
src/gallium/auxiliary/gallivm/lp_bld
On 09/30/2014 01:26 PM, Mathias Fröhlich wrote:
Hi,
On Tuesday, September 30, 2014 13:17:31 Brian Paul wrote:
Same failure with LLVM 3.3.1 too.
Ok, that's what I did not try.
Sorry. I will try to followup immediately ...
Thanks, Mathias. But I'm about to post a patch that fixes things for
Hi,
On Tuesday, September 30, 2014 13:17:31 Brian Paul wrote:
> Same failure with LLVM 3.3.1 too.
Ok, that's what I did not try.
Sorry. I will try to followup immediately ...
Greetings
Mathias
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
h
On 09/30/2014 01:16 PM, Brian Paul wrote:
On 09/30/2014 01:00 PM, Mathias Fröhlich wrote:
Jose,
On Wednesday, September 24, 2014 12:42:24 Jose Fonseca wrote:
That said, the way we use these things are still a bit in flux. Mathias
has some pending patches. BTW, Mathis, should I submit your p
On 09/30/2014 01:00 PM, Mathias Fröhlich wrote:
Jose,
On Wednesday, September 24, 2014 12:42:24 Jose Fonseca wrote:
That said, the way we use these things are still a bit in flux. Mathias
has some pending patches. BTW, Mathis, should I submit your patches
for making llvmpipe thread safe?
Me
Am 30.09.2014 18:46, schrieb Marek Olšák:
> From: Marek Olšák
>
> Use an array of properties indexed by TGSI_PROPERTY_* definitions.
> ---
> src/gallium/auxiliary/draw/draw_gs.c | 23 -
> src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c | 15 +++---
> src/gallium/auxiliary/tg
Jose,
On Wednesday, September 24, 2014 12:42:24 Jose Fonseca wrote:
> That said, the way we use these things are still a bit in flux. Mathias
> has some pending patches. BTW, Mathis, should I submit your patches
> for making llvmpipe thread safe?
Mesa day for me. I did double check the mesa c
Extension enables doing a multisample buffer resolve and buffer
scaling using a single glBlitFrameBuffer() call. Currently, we
have this extension implemented in BLORP which is only used by
SNB and IVB. This patch implements the extension in meta path
which makes it available to Broadwell.
Impleme
with values specific to Intel hardware.
V2: Define and use gen6_get_sample_map() function to initialize
the variables.
V3: Change the function name to gen6_set_sample_maps() and use
memcpy() to fill in the data.
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_context.c
SampleMap{2,4,8}x variables are used in later patches to implement
EXT_framebuffer_multisample_blit_scaled extension.
V2: Use integer array instead of a string.
Bump up the comment.
V3: Use uint8_t type array.
Signed-off-by: Anuj Phogat
---
src/mesa/main/mtypes.h | 32 +
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/intel_extensions.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 046d2a1..10fe10e 100644
--- a/src/mesa/drivers/dri/i
On Tue, Sep 30, 2014 at 5:22 AM, Samuel Iglesias Gonsálvez
wrote:
> Hello,
>
> I am looking at bug 57439 [0] where it shows an error
> in a piglit test [1] related to textureSize() function happening
> in Intel SandyBridge hardware.
>
> According to SNB's PRM documentation (vol4 part1 page 141), t
On Tuesday, September 30, 2014 10:33:42 AM Daniel Vetter wrote:
> On Tue, Sep 30, 2014 at 01:15:56AM -0700, Kenneth Graunke wrote:
> > Write-back caching cannot be used for buffers being scanned out by the
> > display engine; surfaces used for scan-out must be write-through or
> > uncached. I orig
On Tue, Sep 30, 2014 at 8:50 AM, Ian Romanick wrote:
> On 09/20/2014 07:41 PM, Matt Turner wrote:
>> On Sat, Sep 20, 2014 at 6:40 PM, Chris Forbes wrote:
>>> diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
>>> index 79d2e94..c11ad4f 100644
>>> --- a/src/mesa/main/shaderapi.c
>>
https://bugs.freedesktop.org/show_bug.cgi?id=81680
--- Comment #42 from Benjamin Bellec ---
(In reply to comment #40)
> Created attachment 107124 [details] [review]
> possible fix
>
> Could you please test this patch?
Tested-by: Benjamin Bellec
Your patch fixes the crash.
Tested on Evergreen.
On Tue, Sep 30, 2014 at 9:26 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> I'll need this in radeonsi.
> ---
> src/gallium/auxiliary/util/u_math.h | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/src/gallium/auxiliary/util/u_math.h
> b/src/gallium/auxiliary/util/u_math.h
> inde
On Mon, Sep 29, 2014 at 7:16 PM, Jordan Justen wrote:
>
> On 2014-09-29 16:33:33, Anuj Phogat wrote:
> > SampleMap{2,4,8}x variables are used in later patches to implement
> > EXT_framebuffer_multisample_blit_scaled extension.
> >
> > V2: Use integer array instead of a string.
> > Bump up the
Assuming no regressions on other platforms:
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Roland Scheidegger
Need to unwrap the indirect resource otherwise bad things will happen.
Fixes random crashes and timeouts with piglit's arb_indirect_draw tests.
---
src/gallium/drivers/galahad/glhd_context.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a
From: Roland Scheidegger
---
src/gallium/drivers/galahad/glhd_screen.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/drivers/galahad/glhd_screen.c
b/src/gallium/drivers/galahad/glhd_screen.c
index 5a91077..11ab1a9 100644
--- a/src/gallium/drivers/galahad/glhd_screen.c
+
The big pile of patches I just pushed regresses about 25 piglit tests on
SNB. This fixes the regressions.
Signed-off-by: Jason Ekstrand
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 12
src/mesa/drivers/dri/i965/brw_reg.h| 5 +++--
src/mesa/drivers/dri/i965/brw_
Am 30.09.2014 15:16, schrieb Brian Paul:
> The screen argument isn't actually used by lp_jit_screen_init() at this
> time, but let's move the call so that we pass a valid pointer.
>
> v2: don't leak screen if lp_jit_screen_init() fails.
> ---
> src/gallium/drivers/llvmpipe/lp_screen.c |8
Reviewed-by: Charmaine Lee
From: mesa-dev on behalf of Brian Paul
Sent: Tuesday, September 30, 2014 9:31 AM
To: mesa-dev@lists.freedesktop.org
Subject: [Mesa-dev] [PATCH] tgsi: fix Semantic.Name assignment in
tgsi_transform_input_decl()
Assign th
On Tue, Sep 30, 2014 at 1:14 PM, Tom Stellard wrote:
> On Tue, Sep 30, 2014 at 12:29:52PM -0400, Ilia Mirkin wrote:
>> Perhaps do the same thing as util_bitcount, i.e.
>>
>> #if defined(PIPE_CC_GCC) && (PIPE_CC_GCC_VERSION >= 304)
>> return __builtin_popcountll(n);
>> #else
>> ...
>> #endif
>>
>
On Tue, Sep 30, 2014 at 12:29:52PM -0400, Ilia Mirkin wrote:
> Perhaps do the same thing as util_bitcount, i.e.
>
> #if defined(PIPE_CC_GCC) && (PIPE_CC_GCC_VERSION >= 304)
> return __builtin_popcountll(n);
> #else
> ...
> #endif
>
> Perhaps the gcc version check is no longer necessary, unlikel
On 09/30/2014 06:13 PM, Ian Romanick wrote:
On 09/30/2014 12:28 AM, Tapani Pälli wrote:
Patch fixes failing test in WebGL conformance test
'point-no-attributes' when running Chrome on OpenGL ES.
(Shader program may draw points using constant data in shader.)
No Piglit regressions.
This sounds
https://bugs.freedesktop.org/show_bug.cgi?id=81680
--- Comment #41 from Ernst Sjöstrand ---
No longer crashes after applying the patch here!
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@l
From: Marek Olšák
This is a wrong place to flush caches to say the least.
I don't think we need to flush the instruction caches if we don't patch
shaders with DMA.
---
src/gallium/drivers/radeonsi/si_state_draw.c | 8
1 file changed, 8 deletions(-)
diff --git a/src/gallium/drivers/rad
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 2 --
src/gallium/drivers/radeonsi/si_shader.h | 1 -
2 files changed, 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index e76b969..c5f13be 100644
--- a/src/galliu
From: Marek Olšák
It only needs the constant buffer with clip planes and read-write resources
for the GS->VS ring and streamout. That's 2 pointers.
---
src/gallium/drivers/radeonsi/si_shader.c | 9 -
src/gallium/drivers/radeonsi/si_shader.h | 18 ++
src/gallium/d
From: Marek Olšák
st/mesa has the same flag in its shader key, we don't need to do it
in the driver anymore.
Instead, use TGSI_INTERPOLATE_LOC_SAMPLE, which is what st/mesa sets.
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.h | 1 -
src/gallium/d
From: Marek Olšák
The first compiled shader is sometimes useless, because the key doesn't match
the key for the draw call where it's used.
---
src/gallium/drivers/radeonsi/si_state.c | 16 +++-
1 file changed, 3 insertions(+), 13 deletions(-)
diff --git a/src/gallium/drivers/radeons
From: Marek Olšák
We only support 16 vertex attribs, not 32.
---
src/gallium/drivers/radeonsi/si_shader.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.h
b/src/gallium/drivers/radeonsi/si_shader.h
index d9a89e3..c0e5cf4 100644
--- a
From: Marek Olšák
I hope this is correct.
---
src/gallium/drivers/radeonsi/si_shader.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 69382bd..286014c 100644
--- a/s
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 286014c..4e8f80f 100644
--- a/src/gallium/drivers/radeonsi/si_shader
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 8
src/gallium/auxiliary/tgsi/tgsi_scan.h | 3 ---
src/gallium/drivers/llvmpipe/lp_state_fs.c | 4 +++-
src/gallium/drivers/softpipe/sp_quad_blend.c | 5 ++---
src/gallium/drivers/softpipe/sp_setup.c |
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 24 ++--
src/gallium/drivers/radeonsi/si_shader.h | 10 +-
src/gallium/drivers/radeonsi/si_state.c | 25 +++--
src/gallium/drivers/radeonsi/si_state_draw.c | 8
From: Marek Olšák
Use an array of properties indexed by TGSI_PROPERTY_* definitions.
---
src/gallium/auxiliary/draw/draw_gs.c | 23 -
src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c | 15 +++---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 59 ++--
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 25 +++--
src/gallium/drivers/radeonsi/si_shader.h | 2 ++
src/gallium/drivers/radeonsi/si_state.c | 10 +++---
3 files changed, 16 insertions(+), 21 deletions(-)
diff --git a/src/gallium/drivers/radeonsi
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 8 ++--
src/gallium/drivers/radeonsi/si_shader.h | 6 --
src/gallium/drivers/radeonsi/si_state.c | 5 +
3 files changed, 3 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/ga
On 09/20/2014 06:40 PM, Chris Forbes wrote:
> This series adds all the driver-independent bits for ARB_tessellation_shader.
> It's not quite finished, and there are still a handful of ugly hacks to
> remove, but I think it's complete enough to start getting some review
> feedback.
Patches 1, 2, a
On 09/20/2014 06:41 PM, Chris Forbes wrote:
> From: Fabian Bieler
>
> ---
> src/glsl/builtin_variables.cpp | 62
> +-
> src/glsl/glcpp/glcpp-parse.y | 3 ++
> 2 files changed, 64 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/builtin_variables
On 09/20/2014 06:40 PM, Chris Forbes wrote:
> From: Fabian Bieler
>
> ---
> src/mesa/main/mtypes.h| 15 ++-
> src/mesa/program/prog_print.c | 4
> 2 files changed, 18 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
> index
Assign the sem_name parameter, not TGSI_SEMANTIC_GENERIC.
Fixes polygon stipple regression.
---
src/gallium/auxiliary/tgsi/tgsi_transform.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_transform.h
b/src/gallium/auxiliary/tgsi/tgsi_transfor
1 - 100 of 128 matches
Mail list logo
