On Tue, Feb 12, 2013 at 10:45:27PM +0100, Vincent Lejeune wrote:
> It fixes around 100 tfb piglit tests and 16 glean tests.
>
> NOTE: This is a candidate for the Mesa stable branch.
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/AMDILISelDAGToDAG.cpp | 2 ++
>
On Wed, Feb 13, 2013 at 04:34:23PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> The important fix is that the constant interpolation value is stored in the
> parameter slot P0, which is encoded as 2.
>
> In addition, pass the parameter slot as an operand to V_INTERP_MOV_F32
> instead of
_MOV_F32
> instead of hardcoding it there, and add a special register class for the
> parameter slots for type checking and pretty dumping.
>
> NOTE: This is a candidate for the Mesa stable branch.
>
> Signed-off-by: Michel Dänzer
Reviewed-by: Tom Stellard
> ---
>
>
On Wed, Feb 13, 2013 at 10:18:23AM +0100, Christian König wrote:
> Am 13.02.2013 01:20, schrieb Tom Stellard:
> > On Tue, Feb 12, 2013 at 06:13:19PM +0100, Christian König wrote:
> >> From: Christian König
> >>
> > SIInstrFormats.td should contain the instruc
On Wed, Feb 13, 2013 at 10:34:12AM +0100, Christian König wrote:
> Am 13.02.2013 01:39, schrieb Tom Stellard:
> > [SNIP]
> >
> > Way back when I first started working on the backend I was using
> > immediate operands in instructions defined to only uses registers, an
On Tue, Feb 12, 2013 at 06:13:24PM +0100, Christian König wrote:
> From: Christian König
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstrInfo.td|4
> lib/Target/R600/SIInstructions.td |5 +
> 2 files changed, 9 insertions(+)
>
> diff --git a/lib/Target/R6
There's just the one cleanup on patch 10 that you mentioned, but
otherwise the series looks good to me. Should we mark all these patches
as candidates for the stable branch?
For the series:
Reviewed-by: Tom Stellard
On Tue, Feb 12, 2013 at 06:13:13PM +0100, Christian König wrote:
&
On Wed, Feb 13, 2013 at 11:24:24AM -0500, Tom Stellard wrote:
> On Tue, Feb 12, 2013 at 06:13:24PM +0100, Christian König wrote:
> > From: Christian König
> >
> > Signed-off-by: Christian König
> > ---
> > lib/Target/R600/SIInstrInfo.td|4
>
From: Tom Stellard
In some cases, we were losing track of live implicit registers which
was creating dead defs and causing the scheduler to produce invalid
code.
NOTE: This is a candidate for the Mesa stable branch.
---
lib/Target/R600/AMDGPUIndirectAddressing.cpp | 35
From: Tom Stellard
Kernel function arguments are lowered to loads from the PARAM_I address
space. When creating these load instructions, we were initializing
their MachinePointerInfo with an Arguement object that was not attached
to any function. This was causing the MachineScheduler to crash
From: Tom Stellard
This stops the Machine Verifier from complaining about uses of undefined
physical registers.
NOTE: This is a candidate for the Mesa stable branch.
---
lib/Target/R600/R600RegisterInfo.cpp |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/lib
From: Tom Stellard
NOTE: This is a candidate for the Mesa stable branch.
---
lib/Target/R600/R600RegisterInfo.td |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/Target/R600/R600RegisterInfo.td
b/lib/Target/R600/R600RegisterInfo.td
index 3812eb7..a7d847a 100644
single VLIW group).
Also it tries to reduce clause switching by grouping instruction of the
same kind (ALU/FETCH/CF) together.
Vincent Lejeune:
- Support for VLIW4 Slot assignement
- Recomputation of ScheduleDAG to get more parallelism opportunities
Tom Stellard:
- Fix assertion failure when trying
On Wed, Feb 13, 2013 at 04:34:24PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> Requires corresponding LLVM R600 backend fix to work correctly, but even
> without that it doesn't hang anymore.
>
> 13 more little piglits.
>
> NOTE: This is a candidate for the 9.1 branch.
Since this cha
From: Tom Stellard
This way llvm_wrapper.cpp is compiled with -DHAVE_LLVM=0x
---
src/gallium/drivers/r600/Makefile.am | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/Makefile.am
b/src/gallium/drivers/r600/Makefile.am
index 6f48b56..e1c8574
From: Tom Stellard
---
src/gallium/drivers/r600/llvm_wrapper.cpp | 4
src/gallium/drivers/radeon/radeon_llvm_emit.cpp | 9 -
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/llvm_wrapper.cpp
b/src/gallium/drivers/r600/llvm_wrapper.cpp
s for these.
For the series:
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/SIInstructions.td | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/lib/Target/R600/SIInstructions.td
> b/lib/Target/R600/SIInstructions.td
> index d4f7b37..ad1eff7 100644
> --- a/
On Thu, Feb 14, 2013 at 06:07:16PM +0100, Michel Dänzer wrote:
> On Mit, 2013-02-13 at 17:51 +0100, Christian König wrote:
> > Am 13.02.2013 17:07, schrieb Michel Dänzer:
> > > From: Michel Dänzer
> > >
> > > The important fix is that the constant interpolation value is stored in
> > > the
> > >
On Thu, Feb 14, 2013 at 05:47:11PM +0100, Vincent Lejeune wrote:
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/R600RegisterInfo.td | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/Target/R600/R600RegisterInfo.td
> b/lib/Target/R600/R
On Thu, Feb 14, 2013 at 05:47:12PM +0100, Vincent Lejeune wrote:
Reviewed-by: Tom Stellard
> NOTE: This is a candidate for the Mesa stable branch.
> ---
> lib/Target/R600/MCTargetDesc/R600MCCodeEmitter.cpp | 3 +-
> lib/Target/R600/R600Instructions.td
; >
> > NOTE: This is a candidate for the Mesa stable branch.
> >
> > Signed-off-by: Michel Dänzer
> Reviewed-by: Christian König
Reviewed-by: Tom Stellard
>
> > ---
> >
> > v3: Use operand class instead of register class, drop SI_INTERP_CONST
> >
On Thu, Feb 14, 2013 at 06:34:14PM +0100, Christian König wrote:
> From: Christian König
>
> Signed-off-by: Christian König
For the series:
Reviewed-by: Tom Stellard
Don't forget to mark these as candidates for the stable branch.
> ---
> lib/Target/R600/AMDGPUStruc
On Thu, Feb 14, 2013 at 05:48:33PM +0100, Vincent Lejeune wrote:
For the series:
Reviewed-by: Tom Stellard
> NOTE: This is a candidate for the Mesa stable branch.
> ---
> src/gallium/drivers/r600/r600_llvm.c| 6 +-
> src/gallium/drivers/radeon/radeon_setup
On Fri, Feb 15, 2013 at 03:00:24PM +0400, Vadim Girlin wrote:
> On 02/14/2013 02:42 PM, Christian König wrote:
> >Hi Vadim,
> >
> >nice work, I think you've made quite a progress here, but on the other
> >hand it should be clear that the LLVM backend is the future and we
> >should concentrate on th
On Sat, Feb 16, 2013 at 12:51:34PM -0800, Matt Turner wrote:
> On Wed, Feb 13, 2013 at 1:43 PM, Tom Stellard wrote:
> > From: Tom Stellard
> >
> > ---
>
> Candidate for the stable branch as well?
No, only LLVM 3.2 is supported
On Mon, Feb 18, 2013 at 05:27:29PM +0100, Vincent Lejeune wrote:
> Maintaining CONST_COPY Instructions until Pre Emit may prevent some ifcvt case
> and taking them in account for scheduling is difficult for no real benefit.
> ---
> lib/Target/R600/AMDGPU.h| 1 -
> lib/Target/R600
ions to be packed together in a single VLIW group).
> Also it tries to reduce clause switching by grouping instruction of the
> same kind (ALU/FETCH/CF) together.
>
> Vincent Lejeune:
> - Support for VLIW4 Slot assignement
> - Recomputation of ScheduleDAG to get more parallelism op
On Mon, Feb 18, 2013 at 05:27:25PM +0100, Vincent Lejeune wrote:
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/R600Instructions.td | 8
> test/CodeGen/R600/fdiv.v4f32.ll | 8
> 2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git
On Mon, Feb 18, 2013 at 05:27:26PM +0100, Vincent Lejeune wrote:
> mayLoad complexify scheduling and does not bring any usefull info
> as the location is not writeable at all.
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/R600Instructions.td | 2 +-
> 1 file changed, 1
On Mon, Feb 18, 2013 at 05:27:27PM +0100, Vincent Lejeune wrote:
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/AMDILISelDAGToDAG.cpp | 29 +
> 1 file changed, 29 insertions(+)
>
> diff --git a/lib/Target/R600/AMDILISelDAGToDAG.cpp
>
On Mon, Feb 18, 2013 at 05:27:28PM +0100, Vincent Lejeune wrote:
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/R600Instructions.td | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Target/R600/R600Instructions.td
> b/lib/Target/R600/R600Instructions.td
> i
Hi Christian,
>From now on can you cc llvm-comm...@cs.uiuc.edu when you submit a patch.
Thanks,
Tom
On Tue, Feb 19, 2013 at 02:54:23PM +0100, Christian König wrote:
> From: Christian König
>
> Those two files got mixed up.
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstrF
On Wed, Feb 20, 2013 at 12:13:43PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> NOTE: This is a candidate for the Mesa stable branch.
>
> Signed-off-by: Michel Dänzer
Reviewed-by: Tom Stellard
> ---
>
> Not sure why the lack of this didn't seem to cause
From: Tom Stellard
---
.../state_trackers/clover/llvm/invocation.cpp | 47 ---
1 files changed, 39 insertions(+), 8 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 0bd8e22
t; This is a candidate for the 9.1 branch.
>
> Signed-off-by: Alex Deucher
Tested-by: Tom Stellard
> ---
This also fixes a lockup with Civ4 and wine on my Juniper.
-Tom
> src/gallium/drivers/r600/evergreen_state.c |5 -
> 1 files changed, 4 insertions(+), 1 deletions(
With the changes Michel suggested, this series is:
Reviewed-by: Tom Stellard
Are any of these candidates for the stable branch?
-Tom
On Tue, Feb 19, 2013 at 02:54:23PM +0100, Christian König wrote:
> From: Christian König
>
> Those two files got mixed up.
>
> Signed-of
On Thu, Feb 21, 2013 at 09:06:36AM -0500, Tom Stellard wrote:
> With the changes Michel suggested, this series is:
>
> Reviewed-by: Tom Stellard
>
> Are any of these candidates for the stable branch?
>
Please ignore this, I overlooked v2 of this series. I will review t
On Wed, Feb 20, 2013 at 06:47:00PM +0100, Christian König wrote:
> From: Christian König
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstructions.td | 14 +-
> 1 file changed, 5 insertions(+), 9 deletions(-)
>
> diff --git a/lib/Target/R600/SIInstructions.td
> b
For the series:
Reviewed-by: Tom Stellard
On Wed, Feb 20, 2013 at 06:46:47PM +0100, Christian König wrote:
> From: Christian König
>
> Those two files got mixed up.
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SII
Which revisions of Clang and LLVM are you using?
-Tom
> --Aaron
>
> On Wed, Feb 20, 2013 at 4:27 PM, Tom Stellard wrote:
>
> > From: Tom Stellard
> >
> > ---
> > .../state_trackers/clover/llvm/invocation.cpp | 47
> > -
On Mon, Feb 18, 2013 at 05:27:28PM +0100, Vincent Lejeune wrote:
> ---
This fixes the for-loop.cl piglit test, so I've pushed this patch and
backported it to the stable branch.
-Tom
> lib/Target/R600/R600Instructions.td | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Target/R600/
On Thu, Feb 21, 2013 at 10:02:34AM -0600, Aaron Watry wrote:
> On Thu, Feb 21, 2013 at 8:33 AM, Tom Stellard wrote:
>
> > On Thu, Feb 21, 2013 at 08:25:20AM -0600, Aaron Watry wrote:
> > > Hi Tom,
> > >
> > > Mesa+Clover does indeed build against master l
On Fri, Feb 22, 2013 at 05:02:12PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
>
> Signed-off-by: Michel Dänzer
> ---
> lib/Target/R600/SIInstrFormats.td | 24
> lib/Target/R600/SIInstrInfo.td| 22 ++
> lib/Target/R600/SIInstructions.td
This patch has my r-b, but I think we need a better solution for representing
inline constants. See my comment below.
On Fri, Feb 22, 2013 at 05:02:13PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> Use LDS for calculating the deltas between neighbouring pixels. Not sure the
> sign of t
On Fri, Feb 22, 2013 at 05:02:10PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
>
> Signed-off-by: Michel Dänzer
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/SIInstructions.td | 7 ++-
> lib/Target/R600/SIIntrinsics.td | 1 +
> 2 files changed, 7 i
On Fri, Feb 22, 2013 at 05:02:11PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
>
> Signed-off-by: Michel Dänzer
Reviewed-by: Tom Stellard
> ---
> lib/Target/R600/SIInstrInfo.cpp | 28 +++-
> 1 file changed, 27 insertions(+), 1 deletion(-)
On Fri, Feb 22, 2013 at 05:02:37PM +0100, Michel Dänzer wrote:
> From: Michel Dänzer
>
> The number of coordinates to pack will be useful for other address parameters
> as well.
>
> Signed-off-by: Michel Dänzer
These patches depend on LLVM changes, and I would like to track this somehow.
Do yo
From: Tom Stellard
v2:
- Fix order that the clang libraries are passed to the linker to avoid
missing symbol errors.
---
.../state_trackers/clover/llvm/invocation.cpp | 47 ---
src/gallium/targets/opencl/Makefile.am |2 +-
2 files changed, 40
I've sent a v2 of this patch that should fix this. The dependencies between
clang libraries changed so I had to change the order that they were passed to
the linker.
-Tom
> --Aaron
>
> On Wed, Feb 20, 2013 at 4:27 PM, Tom Stellard wrote:
>
> > From: Tom Stellard
>
On Mon, Feb 25, 2013 at 02:13:59PM +, Alle Mahesh wrote:
> Hi,
>
>
> We have run the Khronos test suite on for OpenGLES2.0. There are few failures
> in the GL tests because of the difference in the images generated.
>
> We have found that the images generated using our development board ar
Hi,
I've generated a patch from the master branch of my llvm tree
(http://cgit.freedesktop.org/~tstellar/llvm/) that can be applied against
the LLVM 3.2 code base to add the R600 backend.
For the Mesa 9.1 release, this patch is required in order to use the
radeonsi driver, experimental compute su
On Fri, Mar 01, 2013 at 05:55:31PM +0100, Marek Olšák wrote:
Patches 2-4 are:
Reviewed-by: Tom Stellard
Thanks for working on this, it is a nice improvement.
-Tom
> ---
> src/gallium/drivers/r600/compute_memory_pool.c|1 +
> src/gallium/drivers/r600/evergreen_
On Tue, Mar 05, 2013 at 03:27:19PM +0100, Christian König wrote:
> From: Christian König
>
> Signed-off-by: Christian König
This series has my r-b, but I'd like to test it on r600,
before you push it.
-Tom
> ---
> src/gallium/drivers/radeonsi/radeonsi_shader.c | 31
> +
On Tue, Mar 05, 2013 at 07:51:02PM +0100, Tom Stellard wrote:
> On Tue, Mar 05, 2013 at 03:27:19PM +0100, Christian König wrote:
> > From: Christian König
> >
> > Signed-off-by: Christian König
>
> This series has my r-b, but I'd like to test it on r600,
>
le some shaders with the
> default scheduler because it ends up trying to spill registers, which
> we don't support yet (and which is probably a bad idea in general for
> performance if it can be avoided).
>
> NOTE: This is a candidate for the Mesa stable branch.
>
Revi
From: Tom Stellard
---
src/gallium/drivers/r600/Makefile.am|6 +--
src/gallium/drivers/r600/Makefile.sources |1 -
src/gallium/drivers/r600/evergreen_compute.c|6 +-
src/gallium/drivers/r600/llvm_wrapper.cpp | 61 --
src/gallium
From: Tom Stellard
---
src/gallium/drivers/radeon/radeon_llvm_util.cpp |7 +--
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_llvm_util.cpp
b/src/gallium/drivers/radeon/radeon_llvm_util.cpp
index 5f63ed7..fcfb6d8 100644
--- a/src
From: Tom Stellard
---
src/gallium/drivers/radeon/Makefile.am |1 +
src/gallium/drivers/radeon/radeon_llvm_util.cpp | 21 ++---
2 files changed, 15 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeon/Makefile.am
b/src/gallium/drivers/radeon
From: Tom Stellard
Just delete unused kernels rather than marking them as internal and
running the GlobalDCE pass.
Also implement this function in C and inline it into
radeon_llvm_get_kernel_module()
---
src/gallium/drivers/radeon/radeon_llvm_util.cpp | 26 --
src/gallium
From: Tom Stellard
Also make the function static since it is not used anywhere else.
---
src/gallium/drivers/radeon/radeon_llvm_util.cpp | 18 +++---
src/gallium/drivers/radeon/radeon_llvm_util.h |1 -
2 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/src
From: Tom Stellard
All the functions in this file are now implemented in C.
---
src/gallium/drivers/radeon/Makefile.sources |6 +-
src/gallium/drivers/radeon/radeon_llvm_util.c | 49 ++
src/gallium/drivers/radeon/radeon_llvm_util.cpp | 62
From: Tom Stellard
Signed-off-by: Tom Stellard
---
src/gallium/drivers/radeon/radeon_llvm_util.c | 26 +
src/gallium/drivers/radeon/radeon_llvm_util.h | 26 +
2 files changed, 52 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers
On Fri, Mar 08, 2013 at 09:11:48AM -0600, Aaron Watry wrote:
> LLVM moved a bunch of IR-related headers for version 3.3.
>
> This fixes the libclc build to follow suit.
>
A fix for this has been sitting on the libclc mailing list for a weeks.
I will try to update my branch.
-Tom
> ---
> utils
From: Tom Stellard
This target string now contains four values instead of three. The old
processor field (which was really being interpreted as arch) has been split
into two fields: processor and arch. This allows drivers to pass a
more a more detailed description of the hardware to compiler
From: Tom Stellard
---
This patch depends on the series I posted yesterday that moves
llvm_wrapper.cpp into the radeon directory and re-implements it
using the C API. It also requires the patches on the llvm-commits
list that add support for buffer stores.
src/gallium/drivers/radeon
From: Tom Stellard
---
configure.py | 119 -
1 files changed, 75 insertions(+), 44 deletions(-)
diff --git a/configure.py b/configure.py
index d861c24..dfd9a8f 100755
--- a/configure.py
+++ b/configure.py
@@ -68,6 +68,15 @@ llvm_clang
From: Tom Stellard
This target string now contains four values instead of three. The old
processor field (which was really being interpreted as arch) has been split
into two fields: processor and arch. This allows drivers to pass a
more a more detailed description of the hardware to compiler
From: Tom Stellard
v2:
- Only dump shaders when env variable is set.
---
src/gallium/drivers/radeon/radeon_llvm_util.c |2 +-
src/gallium/drivers/radeon/radeon_llvm_util.h |2 +
src/gallium/drivers/radeonsi/Makefile.sources |1 +
src/gallium/drivers/radeonsi
ected
last year was that the ideas page was not up to par, so if we want to
participate in Google Summer of Code this year, it is important we
have a good ideas page with lots of ideas.
Thanks,
Tom Stellard
___
mesa-dev mailing list
mes
From: Tom Stellard
This target string now contains four values instead of three. The old
processor field (which was really being interpreted as arch) has been split
into two fields: processor and arch. This allows drivers to pass a
more a more detailed description of the hardware to compiler
From: Tom Stellard
This function is a holdover from r600g and is identical to
si_pm4_inval_texture_cache(), so it is not needed.
---
src/gallium/drivers/radeonsi/radeonsi_pm4.c |6 --
src/gallium/drivers/radeonsi/radeonsi_pm4.h |1 -
src/gallium/drivers/radeonsi/si_state_draw.c
From: Tom Stellard
---
src/gallium/drivers/radeonsi/radeonsi_pm4.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_pm4.c
b/src/gallium/drivers/radeonsi/radeonsi_pm4.c
index 9a884f7..4ea30f6 100644
--- a/src/gallium/drivers/radeonsi
From: Tom Stellard
v2:
- Only dump shaders when env variable is set.
v3:
- Don't emit VGT registers
---
src/gallium/drivers/radeon/radeon_llvm_util.c |2 +-
src/gallium/drivers/radeon/radeon_llvm_util.h |2 +
src/gallium/drivers/radeonsi/Makefile.sources |1 +
src/ga
On Wed, Mar 13, 2013 at 09:12:41PM +0100, Vincent Lejeune wrote:
> ---
> lib/Target/R600/AMDILISelDAGToDAG.cpp| 34 ++
> lib/Target/R600/R600InstrInfo.cpp| 54 ++
> lib/Target/R600/R600InstrInfo.h | 3 ++
> lib/Target/R600/R600MachineScheduler.
On Wed, Mar 13, 2013 at 10:26:38PM +0100, Vincent Lejeune wrote:
> ---
> lib/Target/R600/R600ISelLowering.cpp | 23 +++
> test/CodeGen/R600/clamp-constants.ll | 20
> 2 files changed, 43 insertions(+)
> create mode 100644 test/CodeGen/R600/clamp-constants.
my other patches, I'll
submit it upstream.
>
> --Aaron
>
>
> On Tue, Mar 12, 2013 at 3:20 PM, Tom Stellard wrote:
>
> > From: Tom Stellard
> >
> > ---
> > configure.py | 119
> > -
&
On Wed, Mar 13, 2013 at 02:59:07PM -0700, Vincent Lejeune wrote:
> I fixed the coding style issue.
> The include was a debug leftover line, it shouldn't be there.
>
Reviewed-by: Tom Stellard
>
> - Mail original -
> > De : Tom Stellard
> > À : Vin
On Thu, Mar 14, 2013 at 04:49:35PM +0100, Christian König wrote:
> Hi,
>
> as promised here is the mesa part of my indirect addressing patchset for
> radeonsi.
>
> Based on the TGSI changes I've send out earlier today.
>
For the series:
Reviewed-by: Tom Stellard
&g
On Thu, Mar 14, 2013 at 09:12:57PM -0500, Aaron Watry wrote:
> Adds this function for both int and floating data types.
> ---
> generic/include/clc/clc.h |2 ++
> generic/include/clc/integer/max.h |2 ++
> generic/include/clc/integer/max.inc |1 +
> generic/include/clc/math
orrectly in the
backend.
For the series:
Reviewed-by: Tom Stellard
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Mon, Mar 18, 2013 at 01:45:26PM -0700, Ian Romanick wrote:
> On 03/13/2013 10:11 AM, Tom Stellard wrote:
> > Hi,
> >
> > It's time again for Google Summer of Code, so we need to start updating
> > the X.Org ideas page (http://www.x.org/wiki/SummerOfCodeIdeas)
On Sat, Mar 16, 2013 at 05:45:31PM +0100, Vincent Lejeune wrote:
> It allows the backend to generate reads to constant cache which
> are faster that VTX_READ.
Hi Vincent,
Is there a corresponding Mesa change that goes with this patch?
-Tom
> ---
> lib/Target/R600/R600ISelLowering.cpp | 2 +-
>
On Mon, Mar 18, 2013 at 03:07:06PM -0700, Jordan Justen wrote:
> On Wed, Mar 13, 2013 at 10:11 AM, Tom Stellard wrote:
> > - A brief description of the project
> > - A difficulty rating (e.g. easy, medium, hard)
> > - The skills / programming languages required
>
> GL
On Sat, Mar 23, 2013 at 12:37:38PM -0500, Aaron Watry wrote:
> This implementation does a lot of bit shifting and masking. Suffice to say,
> this is somewhat suboptimal... but it does look to produce correct results
> (after the piglit tests were corrected for sign extension issues).
>
> Someone w
On Thu, Mar 28, 2013 at 12:40:18AM +0100, Vincent Lejeune wrote:
> ---
Just a few style issues, with those changes, this patch is:
Reviewed-by: Tom Stellard
> lib/Target/R600/MCTargetDesc/R600MCCodeEmitter.cpp | 169
> +
> lib/Target/R600/R600Ins
On Thu, Mar 28, 2013 at 12:40:19AM +0100, Vincent Lejeune wrote:
> ---
Thanks for working on this, it is a very nice improvement. See my
comments inline.
> lib/Target/R600/AMDGPU.h | 1 +
> lib/Target/R600/AMDGPUTargetMachine.cpp| 1 +
> lib/Target/R600
For the series:
Reviewed-by: Tom Stellard
When you commit these can you update drivers/radeon/LLVM_REVISION.txt
with the revision number of the llvm commits these patches depend on?
Thanks,
Tom
On Thu, Mar 28, 2013 at 12:40:52AM +0100, Vincent Lejeune wrote:
> ---
> src/gallium/driver
On Fri, Mar 29, 2013 at 12:32:19PM +, Mike Lothian wrote:
> Sorry that patch doesn't fix the build
>
> diff -Naur a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
On Sun, Mar 31, 2013 at 02:26:00PM +0200, Armin K. wrote:
> Dana 31.3.2013 14:21, Mike Lothian je napisao:
> >I've already posted this
> >
> >Clover seems to be working as well as it normally does
> >
> >
>
> Doh, and I thought I looked at mesa-dev for a patch. It's hard to
> find it on such high
On Thu, Mar 28, 2013 at 06:47:37PM +0100, Vincent Lejeune wrote:
> ---
Don't forget about the change suggested by Vadim.
Reviewed-by: Tom Stellard
> lib/Target/R600/AMDGPU.h | 1 +
> lib/Target/R600/AMDGPUTargetMachine.cpp| 1 +
>
On Mon, Apr 01, 2013 at 05:42:43PM +0200, Vincent Lejeune wrote:
> ---
Reviewed-by: Tom Stellard
> lib/Target/R600/AMDGPU.h | 1 +
> lib/Target/R600/AMDGPUTargetMachine.cpp| 1 +
> lib/Target/R600/MCTargetDesc/R600MCCodeEmitter.cpp |
From: Tom Stellard
Building libradeonllvm as a shared object has led to a number of bugs
and build system complications, and I don't think it's necessary for
such a small library.
This library was originally changed to a shared object to work around
linker error in egl_static.so,
On Tue, Apr 02, 2013 at 01:38:07PM -0700, Matt Turner wrote:
> ---
Nice catch, will this change have any affect on the compiled code?
-Tom
> src/mesa/program/register_allocate.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/mesa/program/register_allocate.c
From: Marek Olšák
Ported from r600g commit:
8891b2f9c91b2f6c8625184c23a10b8e55875dc0
NOTE: This is a candidate for the stable branches.
---
src/gallium/drivers/radeonsi/r600_blit.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers/radeonsi
From: Tom Stellard
Instead of emitting configuration values (e.g. number of gprs used) in a
predefined order, the LLVM backend now emits these values in
register/value pairs. The first dword contains the register address and
the second dword contians the value to write.
---
src/gallium/drivers
From: Tom Stellard
---
src/gallium/drivers/r600/r600_llvm.c| 8 +--
src/gallium/drivers/radeon/radeon_llvm_emit.cpp | 12 +-
src/gallium/drivers/radeon/radeon_llvm_emit.h | 11 +++--
src/gallium/drivers/radeonsi/radeonsi_shader.c | 31 +
4
From: Tom Stellard
---
configure.ac| 2 ++
src/gallium/drivers/radeon/Makefile.am | 3 +-
src/gallium/drivers/radeon/radeon_llvm_emit.cpp | 38 ++---
3 files changed, 38 insertions(+), 5 deletions(-)
diff --git a/configure.ac b
From: Tom Stellard
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
index 218997f..0aeecc2 100644
--- a/src
From: Tom Stellard
---
lib/Target/R600/AMDGPUAsmPrinter.cpp | 10 +-
test/CodeGen/R600/elf.ll | 1 +
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/lib/Target/R600/AMDGPUAsmPrinter.cpp
b/lib/Target/R600/AMDGPUAsmPrinter.cpp
index f600144..011c602 100644
From: Tom Stellard
Instead of emitting config values in a predefined order, the code
emitter will now emit a 32-bit register index followed by the 32-bit
config value.
---
lib/Target/R600/AMDGPUAsmPrinter.cpp | 19 ---
lib/Target/R600/SIDefines.h | 22
401 - 500 of 1703 matches
Mail list logo
