Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 11:55:25AM -0500, Aaron Watry wrote: > On Thu, Jul 3, 2014 at 10:59 AM, Alex Deucher wrote: > > Someone mentioned stability issues with cedar with the golden register > > kernel patch. Can you see if skipping the golden register setup > > helps? If so can you narrow down

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Aaron Watry
On Thu, Jul 3, 2014 at 10:59 AM, Alex Deucher wrote: > Someone mentioned stability issues with cedar with the golden register > kernel patch. Can you see if skipping the golden register setup > helps? If so can you narrow down which registers are problematic? I'll give it a shot and see if it h

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Aaron Watry
On Thu, Jul 3, 2014 at 11:29 AM, Tom Stellard wrote: > On Thu, Jul 03, 2014 at 10:56:24AM -0400, Tom Stellard wrote: >> On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: >> > Previously, we were assuming that kernel metadata nodes only had 1 operand. >> > >> > Kernels which have attribu

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands\

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 11:59:00AM -0400, Alex Deucher wrote: > On Thu, Jul 3, 2014 at 11:46 AM, Aaron Watry wrote: > > On Thu, Jul 3, 2014 at 9:56 AM, Tom Stellard wrote: > >> On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: > >>> Previously, we were assuming that kernel metadata nod

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Tom Stellard
On Thu, Jul 03, 2014 at 10:56:24AM -0400, Tom Stellard wrote: > On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: > > Previously, we were assuming that kernel metadata nodes only had 1 operand. > > > > Kernels which have attributes can have more than 1, e.g.: > > !0 = metadata !{void (i

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Alex Deucher
On Thu, Jul 3, 2014 at 11:46 AM, Aaron Watry wrote: > On Thu, Jul 3, 2014 at 9:56 AM, Tom Stellard wrote: >> On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: >>> Previously, we were assuming that kernel metadata nodes only had 1 operand. >>> >>> Kernels which have attributes can have

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Aaron Watry
On Thu, Jul 3, 2014 at 9:56 AM, Tom Stellard wrote: > On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: >> Previously, we were assuming that kernel metadata nodes only had 1 operand. >> >> Kernels which have attributes can have more than 1, e.g.: >> !0 = metadata !{void (i32 addrspace(1

Re: [Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-03 Thread Tom Stellard
On Wed, Jul 02, 2014 at 04:34:24PM -0500, Aaron Watry wrote: > Previously, we were assuming that kernel metadata nodes only had 1 operand. > > Kernels which have attributes can have more than 1, e.g.: > !0 = metadata !{void (i32 addrspace(1)*)* @testKernel, metadata !1} > !1 = metadata !{metadata

[Mesa-dev] [PATCH] radeon/llvm: Allocate space for kernel metadata operands

2014-07-02 Thread Aaron Watry
Previously, we were assuming that kernel metadata nodes only had 1 operand. Kernels which have attributes can have more than 1, e.g.: !0 = metadata !{void (i32 addrspace(1)*)* @testKernel, metadata !1} !1 = metadata !{metadata !"work_group_size_hint", i32 4, i32 1, i32 1} Attempting to get the ke