;
+ vl_csc_get_matrix(vl_std,camp, 1, matrix);
+ memcpy(csc_matrix, matrix, sizeof(float)*12);
return VDP_STATUS_OK;
}
I haven't checked if the matrix are correct and precise enough, but
otherwise looks good to me.
Reviewed by: Christian König deathsim...@vodafone.de
I've just tested and pushed the whole series, thanks for that.
Christian.
On 31.10.2011 18:37, Maarten Lankhorst wrote:
Add a struct with all the fields.
Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com
---
Just because vdpau state tracker supports it, doesn't mean anything uses it
Hi Maarten,
On 06.11.2011 10:41, Maarten Lankhorst wrote:
Hey Christian,
On 10/31/2011 11:00 AM, Christian König wrote:
On 31.10.2011 10:23, Maarten Lankhorst wrote:
With the smpte240 profile, which was missing.
Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com
---
src/gallium
On 14.11.2011 12:28, Jose Fonseca wrote:
- Original Message -
Indirect addressing on TEMP should be forbidden and only allowed for
TEMPORARY_ARRAY, I'd very much like to know which values I can keep
in
registers and which ones I have to store to thread-local memory (I
can't
do things
On 15.11.2011 17:52, Maarten Lankhorst wrote:
Deleted:
- begin_frame/end_frame: Was only useful for XvMC, should be folded into flush..
I'm not completely happy with the current interface also, but if you
remove the state tracker ability to control how many buffers are used,
which in turn
On 16.11.2011 15:38, Maarten Lankhorst wrote:
Hey,
On 11/16/2011 02:47 PM, Christian König wrote:
On 15.11.2011 17:52, Maarten Lankhorst wrote:
Deleted:
- begin_frame/end_frame: Was only useful for XvMC, should be folded into flush..
I'm not completely happy with the current interface also
On 26.11.2011 22:23, Maarten Lankhorst wrote:
Hey Andy,
On 11/26/2011 01:44 PM, Andy Furniss wrote:
Maarten Lankhorst wrote:
Testing interlaced videos that decode correctly with nvidia vdpau would help
a lot to figure out what the proper way to handle interlacing would be, so if
someone has
On 11.12.2011 07:13, Younes Manton wrote:
On Tue, Dec 6, 2011 at 4:51 PM, Andy Furnissandy...@ukfsn.org wrote:
Maarten Lankhorst wrote:
The brokenness in vlVdpVideoMixerRender was compensating for
brokenness in vlVdpPresentationQueueDisplay, so fix both at
the same time.
These fix the two
Hi Maarten,
On 12.12.2011 13:25, Maarten Lankhorst wrote:
The problem is destination rectangle can be different for video and
subpictures, this breaks mplayer OSD if you handle it incorrectly.
Hi Andy,
On 12.12.2011 19:15, Andy Furniss wrote:
Christian König wrote:
@Andy: Could you please confirm that they are also working? (They should
apply ontop of current master).
Yes - they are working OK for me.
Ok, so I committed them just a couple of minutes ago.
One thing I noticed
Committed all three with just a minor change:
removed check_video_surface in favor of doing the checks directly in the
render function.
On 19.12.2011 10:10, Maarten Lankhorst wrote:
Currently only validating, since nothing else can be done with it yet
Signed-off-by: Maarten
On 20.12.2011 12:43, Maarten Lankhorst wrote:
And add more sanity checks to stream. This shouldn't break things beyond those
that aren't broken already.
Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com
---
And yes Andy, I mean that I haven't found a good video yet to fix the playback
On 20.12.2011 19:20, Maarten Lankhorst wrote:
Hey Christian,
On 12/20/2011 02:16 PM, Christian König wrote:
Why do you want to change that anyway?
The search for start codes was especially split out of the VLC stuff, because start
codes start are byte aligned anyway and it doesn't make much
version of the patch.
Take a look at the attached one, does it now cover all cases?
On 21.12.2011 17:24, Younes Manton wrote:
2011/12/21 Maarten Lankhorstm.b.lankho...@gmail.com:
Hey Christian,
On 12/21/2011 04:41 PM, Christian König wrote:
Those functions are called a couple of million times
Hi Maarten,
On 21.12.2011 21:52, Maarten Lankhorst wrote:
It would be nice if you inlined patches for easier reviewing. :)
Well I can try, but I can't promise that Thunderbird isn't badly fucking
up all whitespaces, newest version of the patch is in-lined below.
I'm spotting an overflow
Sound it it was a good idea to look into my mailbox today. Thanks, patch
is committed.
Have a nice x-mas, and a happy new year,
Christian.
On 24.12.2011 12:33, Maarten Lankhorst wrote:
Reported-by: Andy Furnissandy...@ukfsn.org
Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com
---
The patch caused some problems with XvMC (try for example skipping
forward/backwards, or just fast forward playing).
So I split it up into three separate patches and reworked the handling of
buffers in the XvMC case, please review.
Christian.
___
Based on patches from Maarten Lankhorst m.b.lankho...@gmail.com
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/vl/vl_decoder.c | 20 +---
src/gallium/auxiliary/vl/vl_decoder.h |9 +--
src/gallium/auxiliary/vl/vl_mpeg12_decoder.c
Looks good on first sign. I'm waiting for this for quite some time now,
cause it makes XvMC state tracker implementation more cleaner and faster.
Acked-by: Christian König deathsim...@vodafone.de
On 02.01.2012 01:22, Marek Olšák wrote:
Please see the diff for further info.
This paves the way
Hi Maarten,
first of all: Happy new Year and sorry for the late reply, have been on
vacation for the last week.
On 29.12.2011 16:41, Maarten Lankhorst wrote:
Hey Christian,
Op 26-12-11 14:00, Christian König schreef:
Based on patches from Maarten Lankhorstm.b.lankho...@gmail.com
Signed
-12-11 14:00, Christian König schreef:
Based on patches from Maarten Lankhorstm.b.lankho...@gmail.com
Signed-off-by: Christian Königdeathsim...@vodafone.de
diff --git a/src/gallium/include/pipe/p_context.h
b/src/gallium/include/pipe/p_context.h
index de79a9b..f7ee522 100644
--- a/src/gallium
Add the infrastructure, but not the decode implementation.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/vl/vl_video_buffer.c | 35 ++-
src/gallium/auxiliary/vl/vl_video_buffer.h |2 +-
src/gallium/include/pipe/p_video_decoder.h
Just like in the rest of gallium, this reduces the
number of parameters significantly.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 27 ++
src/gallium/auxiliary/vl/vl_video_buffer.c | 47
Add a second extened constructor that takes plane
textures for the video buffer. Also provide a
function for texture templates.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/vl/vl_video_buffer.c | 112 +---
src/gallium/auxiliary/vl
No implementation so far, just the defines for
VDPAUs picture info structure.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/include/pipe/p_video_state.h | 40 +++
src/gallium/state_trackers/vdpau/decode.c | 61 +
2 files
Create the video buffers in the format the driver preffers.
This temporary creates problems with decoder less VDPAU video playback.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/drivers/nouveau/nouveau_video.c|2 ++
src/gallium/drivers/nvfx/nvfx_screen.c
Recreate the video buffer in PutBitsYCbCr if the format doesn't match.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/state_trackers/vdpau/surface.c | 54 ++---
src/gallium/state_trackers/vdpau/vdpau_private.h |2 +-
2 files changed, 37
Recreate the video buffer if the decoder can't handle it.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/state_trackers/vdpau/decode.c | 22 +-
1 files changed, 21 insertions(+), 1 deletions(-)
diff --git a/src/gallium/state_trackers/vdpau/decode.c
We actually implemented YV21 instead of YV12, so fix the plane ordering.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 13 +---
src/gallium/auxiliary/vl/vl_video_buffer.c | 37 +++--
src/gallium/auxiliary
Hi Maarten,
On 08.01.2012 14:09, Maarten Lankhorst wrote:
Ok. I wont block it any more then. Do you have a followup patch to get
rid of quant matrix and separate reference pictures too? Or should I
resend it, I kind of lost my rebased git tree, so if you already have
the followup patches it
Hi Marek,
the unused vertex shader inputs where there to work around a bug in
r600g, which I fixed a couple of hours ago.
I have also noticed create_ref_vert_shader doesn't write the vertex
position, which might lead to undefined rendering.
The position should be calculated in calc_position,
Hi,
sorry for the late reply, have been in Canada on a team meeting.
Did you got it working Andy or do you need some more help?
I think the overall situation with finding libraries in the mesa
makefiles is not the best, but that's not prio 1 at the moment.
It should work with a standard
Am Freitag, den 29.07.2011, 18:23 -0400 schrieb Younes Manton:
On Fri, Jul 29, 2011 at 9:37 AM, Maarten Lankhorst
m.b.lankho...@gmail.com wrote:
With some help from the nouveau team I managed to get video acceleration
working for my nv96 card. The video buffer api works well enough for
Hi,
Am Dienstag, den 02.08.2011, 15:00 +0800 schrieb fykc...@gmail.com:
I've tried mesa-7.11rc4 with pipe-video patch on a Loongson3A based notebook:
* CPU: 4-core MIPS64 compatible (Little-endian, 64bit kernel with an
o32 userland)
* Graphics: RS780E
a bit unusual hardware configuration,
Am Donnerstag, den 04.08.2011, 13:41 +0800 schrieb fykc...@gmail.com:
I guess that may be related with driver or DRM stack, after playback,
my desktop corrupts
http://dev.lemote.com/files/upload/software/temp/newmobcal1920-desktop-corrupt.png
(Similar with -vc ffmpeg12vdpau -vo vdpau).
I've
Am Samstag, den 06.08.2011, 14:37 -0400 schrieb Younes Manton:
The attached patch I believe should satisfy everyone's needs here. It
removes the use of pipe_video_decode_buffer from the state tracker and
moves it to the shader decoder. This lets every decoder parse the
incoming macroblocks
Am Montag, den 08.08.2011, 15:00 +0200 schrieb Maarten Lankhorst:
On 08/08/2011 12:10 PM, Christian König wrote:
Most modern players doesn't do it like this any more, but it still seems
to cause a bunch of problems when seeking or fast forward both with
mplayer and xine.
So I would
Am Donnerstag, den 11.08.2011, 12:04 -0400 schrieb Younes Manton:
It's been brought to my attention that the source this is based on is
GPL'd, which means it needs to go before 7.12 is released since it's
incompatible with Mesa's MIT license.
That is actually not so problematic as it sounds in
Am Freitag, den 12.08.2011, 10:49 -0400 schrieb Younes Manton:
Sorry, by incompatible I didn't mean that you couldn't use them
together, but that one is more restrictive than the other. Like the
discussion you quoted states, if you combine MIT and GPL you have to
satisfy both of them, which
Am Dienstag, den 16.08.2011, 01:15 -0400 schrieb Younes Manton:
Anyway, here are some specific comments:
+ for (; num_macroblocks 0; --num_macroblocks) {
+ mb-base.codec = PIPE_VIDEO_CODEC_MPEG12;
+ mb-macroblock_address = xvmc_mb-x +
context-width_in_macroblocks * xvmc_mb-y;
Am Mittwoch, den 17.08.2011, 12:52 +0100 schrieb Andy Furniss:
Maybe this is already known/just not complete yet, but as I've
previously written that r600 -vo vdpau without decode looked OK I ought
to mention it as I've just noticed.
It is just incomplete, width/height is made a multiple of
Hey Andy,
the attached patch should fix your issue.
Please test it.
Christian.
Am Mittwoch, den 17.08.2011, 12:52 +0100 schrieb Andy Furniss:
Maybe this is already known/just not complete yet, but as I've
previously written that r600 -vo vdpau without decode looked OK I ought
to mention
Hi Maarten,
could you put this into SetDecoderStatus instead? This makes the picture
structure also available in end_frame.
Also named initializers shouldn't be used according to the coding rules.
Christian.
Am Sonntag, den 28.08.2011, 01:59 +0200 schrieb Maarten Lankhorst:
picture_structure
Am Montag, den 29.08.2011, 00:36 -0400 schrieb Younes Manton:
[snip]
Well, that was what the last discussion was all about, whether or not
decode buffers should be handled internally by each driver or
externally. It was decided to handle it externally, to simplify life
for drivers that
I haven't really tested this, so please speak up if anybody finds any
regression, but I'm going to push this if there are no objections.
Beside being a real killer for performance, the whole [New Thread
0x7fffe60fb700 (LWP 5212)]/[Thread 0x7fffe60fb700 (LWP 5212) exited] in
gdb is quite annoying.
Am Sonntag, den 28.08.2011, 18:55 +0100 schrieb Andy Furniss:
deathsim...@vodafone.de wrote:
From: Christian Königdeathsim...@vodafone.de
Based on work of Maarten Lankhorst this time.
I am getting new artifacts on r600 and softpipe (ignoring all the other
issues it has) when using -vc
Am Dienstag, den 30.08.2011, 00:11 +0200 schrieb Maarten Lankhorst:
Hey Christian,
On 08/29/2011 10:14 AM, Christian König wrote:
Am Sonntag, den 28.08.2011, 19:13 +0200 schrieb Maarten Lankhorst:
Hey,
On 08/28/2011 05:04 PM, Christian König wrote:
Hi Maarten,
could you put
Ok, tested it a bit (but not to extensive).
Seems to work fine, so pushed it to master.
Thanks,
Christian.
Am Dienstag, den 30.08.2011, 03:25 +0200 schrieb Marek Olšák:
Thanks. I wanted to do this long ago.
Reviewed-by: Marek Olšák mar...@gmail.com
On Sat, Aug 27, 2011 at 10:57 PM,
Am Dienstag, den 30.08.2011, 23:29 +0200 schrieb Maarten Lankhorst:
I think the question we should ask isn't if one driver or another needs
something, but more if one or another behaviour is specific to an
interface or not. So when an interface theoretically allows a behaviour
(even if
Start with correctly defining IA44 and AI44 formats.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/util/u_format.csv |6 +++-
src/gallium/auxiliary/util/u_format_yuv.c | 38 ---
src/gallium/auxiliary/util/u_format_yuv.h | 40
Sampling worked out of the box, but this make them work as surface as well.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/drivers/r600/evergreen_state.c |4
src/gallium/drivers/r600/r600_state.c |4
2 files changed, 8 insertions(+), 0 deletions
This is no longer needed, since we now have native support for IA44 and AI44.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/state_trackers/xorg/xvmc/subpicture.c | 38 +++--
1 files changed, 5 insertions(+), 33 deletions(-)
diff --git a/src/gallium
---
src/gallium/state_trackers/xorg/xvmc/subpicture.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/gallium/state_trackers/xorg/xvmc/subpicture.c
b/src/gallium/state_trackers/xorg/xvmc/subpicture.c
index 6fe4773..52677e2 100644
---
They are mostly used for menu overlay.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/util/u_format.csv |2 ++
src/gallium/include/pipe/p_format.h |6 --
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/util
Prepares for vdpau menu overlay.
---
src/gallium/auxiliary/vl/vl_compositor.c| 84 ---
src/gallium/auxiliary/vl/vl_compositor.h| 22 +-
src/gallium/state_trackers/vdpau/mixer.c|2 +-
src/gallium/state_trackers/vdpau/presentation.c |2 +-
This gets mplayers menu overlay working.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/state_trackers/vdpau/device.c|3 +
src/gallium/state_trackers/vdpau/mixer.c | 13 +-
src/gallium/state_trackers/vdpau/output.c| 258
Am Montag, den 29.08.2011, 23:29 +0100 schrieb Andy Furniss:
Ironically your question made me search to see if I had any other mpeg2
HD not from w6rz - I found one and it has issues with -vf ffmpeg12vdpau :-)
Looks like there is some garbage at the end of the stream, mplayer is
also complaining
Am Donnerstag, den 01.09.2011, 08:09 -0600 schrieb Kevin DeKorte:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 09/01/2011 07:50 AM, Christian König wrote:
This gets mplayers menu overlay working.
Signed-off-by: Christian König deathsim...@vodafone.de ---
src/gallium
Am Donnerstag, den 01.09.2011, 08:14 -0700 schrieb Jose Fonseca:
- Original Message -
- It doesn't look like a typical intensity -- if so then RGB should
have the same swizzle instead of x00, and y00. So either put a
xxx or yyy RGB swizzles, or rename these to
Am Donnerstag, den 01.09.2011, 17:08 +0200 schrieb Christoph Bumiller:
On 01.09.2011 17:02, Younes Manton wrote:
On Thu, Sep 1, 2011 at 10:56 AM, Michel Dänzer mic...@daenzer.net wrote:
On Don, 2011-09-01 at 15:50 +0200, Christian König wrote:
Start with correctly defining IA44 and AI44
...@daenzer.net wrote:
On Don, 2011-09-01 at 15:50 +0200, Christian König wrote:
Start with correctly defining IA44 and AI44 formats.
Signed-off-by: Christian König deathsim...@vodafone.de
---
src/gallium/auxiliary/util/u_format.csv |6 +++-
src/gallium/auxiliary/util
Am Donnerstag, den 01.09.2011, 13:28 -0400 schrieb Younes Manton:
2011/9/1 Christian König deathsim...@vodafone.de:
This gets mplayers menu overlay working.
+ if (destination_rect) {
+ res_tmpl.width0 = abs(destination_rect-x0-destination_rect-x1);
+ res_tmpl.height0 = abs
Am Donnerstag, den 01.09.2011, 19:54 +0100 schrieb Andy Furniss:
Christian König wrote:
Looks like there is some garbage at the end of the stream, mplayer is
also complaining nicely about TS_PARSE: COULDN'T SYNC.
I played around a bit more, it seems that rather than garbage it's
Looks good, I'm going to push it, but one small thing:
Am Freitag, den 02.09.2011, 16:20 +0200 schrieb Maarten Lankhorst:
The preferred solution to keeping track of the picture structure
has been putting it in the state tracker, so use picture_structure
instead of frame_started to check if a
Am Samstag, den 03.09.2011, 18:07 +0200 schrieb Maarten Lankhorst:
You're right, that check is messed up, the ( should be moved up a line.
Thanks for catching that. :)
I fixed the check and pushed it upstream.
Thanks,
Christian.
___
mesa-dev mailing
Am Freitag, den 16.09.2011, 12:58 -0700 schrieb Jose Fonseca:
[SNIP]
This change would be best to describe a vertex and texture fetch as
implemented by Radeons:
diff --git a/src/gallium/auxiliary/util/u_format.h
b/src/gallium/auxiliary/util/u_format.h
index 2eb3e1b..8c4d67f 100644
Sorry for the delay, got no internet for nearly a week.
Patch is committed.
Thanks,
Christian.
Am Donnerstag, den 22.09.2011, 21:03 +0200 schrieb Emeric Grange:
This patch fix a Unresolved Symbols run time error when using G3DVL
through the VDPAU state tracker, by linking the vdpau targets
Hi Romain,
Am Sonntag, den 25.09.2011, 11:11 +0200 schrieb Romain Failliot:
Hi all!
I'm an experienced programmer, but I've never touched the mesa code.
I desperately want to help, but for that, I need a mentor to teach
me the basics about the project, the good practices and eventually
Am Mittwoch, den 05.10.2011, 10:57 -0400 schrieb Tom Stellard:
Reduction instructions can't share an instruction group with other
instructions, so we need to check for them when trying to merge
instruction groups.
Hui? Where do you got that info?
I especially created that code to let a DOT4
Hi Vadim,
wow quite impressive, does it also contains peep-hole optimisation?
I tried to implement that once, but failed because I never got all the
dependencies between opcodes correctly resolved. Rescheduling export
instructions and setting the barrier flag of CF instructions correctly
can
On 25.10.2011 20:57, Maarten Lankhorst wrote:
On 10/25/2011 08:00 PM, Younes Manton wrote:
On Tue, Oct 25, 2011 at 1:35 PM, Maarten Lankhorst
m.b.lankho...@gmail.com wrote:
Is there a reason not to use/add to the code in vl_csc.c in this case?
It supports 601 and 709, wider gamut, and
Am 22.01.2013 13:08, schrieb Michel Dänzer:
On Mon, 2013-01-21 at 22:28 +0100, Christian König wrote:
v2: don't mess up other loops
Signed-off-by: Christian König deathsim...@vodafone.de
Series is
Tested-by: Michel Dänzer michel.daen...@amd.com
No piglit regressions with radeonsi. :)
P.S
On 23.01.2013 03:18, Tom Stellard wrote:
On Wed, Jan 23, 2013 at 02:20:21AM +0100, Christoph Bumiller wrote:
On 23.01.2013 02:07, Vadim Girlin wrote:
On 01/23/2013 04:42 AM, Christoph Bumiller wrote:
On 23.01.2013 01:21, Vadim Girlin wrote:
On 01/23/2013 03:59 AM, Vincent Lejeune wrote:
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
Tested-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 71 +++---
1 file changed, 36 insertions(+), 35 deletions(-)
diff --git
v2: don't mess up other loops
Signed-off-by: Christian König deathsim...@vodafone.de
Tested-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600
Hi Tom,
the following patches should fix the known issues with the structurizer and
also optimize it a bit more.
The first two are already tested by Michel. I tested the last one with radeonsi
and it doesn't seem to regress anything and actually fixes four more piglit
tests.
Please apply to
From: Christian König christian.koe...@amd.com
The loop handling in the CFG structurizer incorrectly assumed
that only BasicBlock nodes can have a back edge, but that is
also possible for the exit edges of subregions.
Fixing 4 more piglit tests on radeonsi.
Signed-off-by: Christian König
Hi guys,
I needed to rearrange the order of patches in this patchset, cause fixing one
bug lead to the discovery of a couple of other more nasty bugs.
It now fixes four piglit tests with radeonsi and also survives the use cases
Tom provided without causing any regression.
So please test again,
From: Christian König christian.koe...@amd.com
Intersecting loop handling was wrong.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 296 ++
1 file changed, 183 insertions(+), 113 deletions(-)
diff --git a/lib
From: Christian König christian.koe...@amd.com
This fixes a couple of bugs and incorrect assumptions,
in total four more piglit tests now pass.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 360 +++---
1 file
From: Christian König christian.koe...@amd.com
Otherwise we sometimes produce invalid code.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 146 +-
1 file changed, 81 insertions(+), 65 deletions(-)
diff --git
Am 01.02.2013 23:58, schrieb Tom Stellard:
On Fri, Feb 01, 2013 at 04:05:51PM +0100, Christian König wrote:
Hi guys,
I needed to rearrange the order of patches in this patchset, cause fixing one
bug lead to the discovery of a couple of other more nasty bugs.
It now fixes four piglit tests
From: Christian König christian.koe...@amd.com
Intersecting loop handling was wrong.
Signed-off-by: Christian König christian.koe...@amd.com
Tested-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 296 ++
1 file changed, 183
From: Christian König christian.koe...@amd.com
Otherwise we sometimes produce invalid code.
Signed-off-by: Christian König christian.koe...@amd.com
Tested-by: Michel Dänzer michel.daen...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 146 +-
1 file changed
From: Christian König christian.koe...@amd.com
This fixes a couple of bugs and incorrect assumptions,
in total four more piglit tests now pass.
v2: fix small bug in the dominator updating
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp
From: Christian König christian.koe...@amd.com
The _SGPR variants where wrong.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUCodeEmitter.h|4 -
lib/Target/R600/AMDILISelDAGToDAG.cpp | 53
lib/Target/R600
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td | 10 ++
lib/Target/R600/SIInstructions.td | 19 ---
lib/Target/R600/SILowerLiteralConstants.cpp |1
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c |9 +
1 file changed, 9 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König christian.koe...@amd.com
Seems to work fine now.
Signed-off-by: Christian König christian.koe...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c |6 --
1 file changed, 6 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src
From: Christian König christian.koe...@amd.com
v2: rebased on current upstream
Signed-off-by: Christian König christian.koe...@amd.com
Reviewed-by: Tom Stellard thomas.stell...@amd.com
---
lib/Target/R600/SIInstrInfo.td | 10 ++
lib/Target/R600/SIInstructions.td
From: Christian König christian.koe...@amd.com
The _SGPR variants where wrong.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUCodeEmitter.h|4 -
lib/Target/R600/AMDILISelDAGToDAG.cpp | 53
lib/Target/R600
From: Christian König christian.koe...@amd.com
With the llvm patches, fixing 14 piglit tests in total.
v2: increase the const limit
Signed-off-by: Christian König christian.koe...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
src/gallium/drivers/radeonsi
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_shader.c |9 +
1 file changed, 9 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König christian.koe...@amd.com
Remove all the unused code.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUAsmPrinter.cpp |2 +-
lib/Target/R600/MCTargetDesc/SIMCCodeEmitter.cpp | 141 +-
lib/Target/R600
Am 11.02.2013 15:35, schrieb Michel Dänzer:
On Son, 2013-02-10 at 19:38 +0100, Vincent Lejeune wrote:
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
index a09f243..7e50e86 100644
--- a/lib/Target/R600/SIInstructions.td
+++ b/lib/Target/R600/SIInstructions.td
From: Christian König christian.koe...@amd.com
Using the new NearestCommonDominator class.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 31 +-
1 file changed, 22 insertions(+), 9 deletions(-)
diff --git
From: Christian König christian.koe...@amd.com
Stop adding more instructions than necessary.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 40 +-
1 file changed, 39 insertions(+), 1 deletion(-)
diff --git
From: Christian König christian.koe...@amd.com
Generate more than one loop if it seems to make sense.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUStructurizeCFG.cpp | 342 +-
1 file changed, 147 insertions(+), 195 deletions
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUCodeEmitter.h| 45
lib/Target/R600/MCTargetDesc/AMDGPUMCCodeEmitter.h |7 ---
2 files changed, 52 deletions(-)
delete mode
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 66 +
lib/Target/R600/SIInstrInfo.td| 66 -
2 files changed, 66
1 - 100 of 1736 matches
Mail list logo