search for: nov26

...audit the macros and change them to UPPERCASE so it's obvious that there's some macro magic at work. > I've made these changes and also renamed the directory from AMDGPU to R600 as suggested by Chris. An updated patch can be found here: http://people.freedesktop.org/~tstellar/llvm/Nov26/ I've also included in that directory the 6 patches I wrote to implement these changes, to make it easier to see what is different. If you see anything I left out, let me know. > Overall I think this should go in as soon as possible. It has been around for a while and you have been active...

...nge them to UPPERCASE so it's obvious that there's some macro magic at work. >> > > I've made these changes and also renamed the directory from AMDGPU > to R600 as suggested by Chris. An updated patch can be found here: > http://people.freedesktop.org/~tstellar/llvm/Nov26/ Sorry for being pedantic, but there are still misplaced braces in your patch. You can easily find them in the source code with: $ grep -r '^[ ]*{$' Otherwise I don't see anything else that needs fixing offhand. > I've also included in that directory the 6 patches I wrote to...

On 01.11.2012, at 14:44, Tom Stellard <tom at stellard.net> wrote: > Moving this thread to llvmdev. > > On Tue, Oct 30, 2012 at 11:09:34PM -0700, Chris Lattner wrote: >> On Oct 30, 2012, at 11:35 AM, Tom Stellard <tom at stellard.net> wrote: >>>> Hi Tom, >>>> >>>> Time is running short, but this would be great. The best place to

Moving this thread to llvmdev. On Tue, Oct 30, 2012 at 11:09:34PM -0700, Chris Lattner wrote: > On Oct 30, 2012, at 11:35 AM, Tom Stellard <tom at stellard.net> wrote: > >> Hi Tom, > >> > >> Time is running short, but this would be great. The best place to start is to begin decomposing the mega-patch into individual pieces that makes sense. Do you have