search for: d2222

On Wed, Nov 20, 2013 at 9:46 AM, Timur Iskhodzhanov <timurrrr at google.com>wrote: > Eric, David, > > 2013/11/19 Timur Iskhodzhanov <timurrrr at google.com>: > > Attached is a slightly updated patch. > > (it doesn't include D2222 yet). > > The new version of the patch stopped fitting into the llvmdev 100K limit, > so I've uploaded it to http://llvm-reviews.chandlerc.com/D2232 > (you need to apply D2222 first if you'd like to give D2232 a try) > > > 2013/11/19 Eric Christopher <echristo at gma...

Eric, David, 2013/11/19 Timur Iskhodzhanov <timurrrr at google.com>: > Attached is a slightly updated patch. > (it doesn't include D2222 yet). The new version of the patch stopped fitting into the llvmdev 100K limit, so I've uploaded it to http://llvm-reviews.chandlerc.com/D2232 (you need to apply D2222 first if you'd like to give D2232 a try) > 2013/11/19 Eric Christopher <echristo at gmail.com>: >> In gene...

Attached is a slightly updated patch. (it doesn't include D2222 yet). 2013/11/19 Eric Christopher <echristo at gmail.com>: > In general I do think we're going to need to abstract it out as much > as possible. I'm not sure what the previous patch looks like, but > abstracting the interface out would be general goodness for this. We > c...

...gt; On Wed, Nov 20, 2013 at 9:46 AM, Timur Iskhodzhanov <timurrrr at google.com> > wrote: >> >> Eric, David, >> >> 2013/11/19 Timur Iskhodzhanov <timurrrr at google.com>: >> > Attached is a slightly updated patch. >> > (it doesn't include D2222 yet). >> >> The new version of the patch stopped fitting into the llvmdev 100K limit, >> so I've uploaded it to http://llvm-reviews.chandlerc.com/D2232 >> (you need to apply D2222 first if you'd like to give D2232 a try) >> >> > 2013/11/19 Eric Christo...

On Wed, Nov 20, 2013 at 9:46 AM, Timur Iskhodzhanov <timurrrr at google.com> wrote: > Eric, David, > > 2013/11/19 Timur Iskhodzhanov <timurrrr at google.com>: >> Attached is a slightly updated patch. >> (it doesn't include D2222 yet). > > The new version of the patch stopped fitting into the llvmdev 100K limit, > so I've uploaded it to http://llvm-reviews.chandlerc.com/D2232 > (you need to apply D2222 first if you'd like to give D2232 a try) > That's a big patch. :) >> 2013/11/19 Eric Chr...

...gt; > On Wed, Nov 20, 2013 at 9:46 AM, Timur Iskhodzhanov <timurrrr at google.com> > wrote: > > Eric, David, > > > > 2013/11/19 Timur Iskhodzhanov <timurrrr at google.com>: > >> Attached is a slightly updated patch. > >> (it doesn't include D2222 yet). > > > > The new version of the patch stopped fitting into the llvmdev 100K limit, > > so I've uploaded it to http://llvm-reviews.chandlerc.com/D2232 > > (you need to apply D2222 first if you'd like to give D2232 a try) > > > > That's a big patch...

I wrote some more lit tests for my patch and realized I was generating some redundant info. This is fixed now. Attached is a new version of the prototype patch with some more tests. 2013/11/15 João Matos <ripzonetriton at gmail.com>: > Hi Timur, > > There's also a pending patch adding CodeView support in Phab: > http://llvm-reviews.chandlerc.com/D165 I haven't looked at

In general I do think we're going to need to abstract it out as much as possible. I'm not sure what the previous patch looks like, but abstracting the interface out would be general goodness for this. We can talk about designs for that as we move on. As far as how to migrate the decision down we can have it both as an option to code gen maybe or, for now, make it dependent upon triple. The