Include this in 3.4? (avoids embarassing crash when init-captures are used w generic lambdas)

[Faisalv]

Thanks - but this was already taken care of by one of the patches richard asked u to pull in (which u did) - sorry about the confusion :)


Sent from my iPhone

On Dec 7, 2013, at 5:52 PM, Bill Wendling <isanbard at gmail.com> wrote:

> Please put the patch in ToT first and then ping me with the revision to add.
> 
> -bw
> 
> On Dec 4, 2013, at 10:40 AM, Richard Smith <richard at metafoo.co.uk> wrote:
> 
>> Sorry for the delay on that patch! Yes, we should try to get that fix into 3.4.
>> 
>> 
>> On Wed, Dec 4, 2013 at 10:22 AM, David Blaikie <dblaikie at gmail.com> wrote:
>> It's best to ping the original review thread rather than creating
>> others. Feel free to highlight that this might be a 3.4 issue, of
>> course.
>> 
>> On Wed, Dec 4, 2013 at 5:16 AM, Faisal Vali <faisalv at gmail.com> wrote:
>> > http://llvm-reviews.chandlerc.com/D2092
>> >
>> > I apologize if I'm being a pest about this patch - but since I'm not sure
>> >
>> > what the cutoff date is for 3.4 (but it seems like it's rapidly approaching)
>> > - and I know Richard had expressed interest in incorporating this patch into
>> > 3.4 -
>> > just thought i'd bring it up again.
>> >
>> > FYI: Currently clang crashes on the following code:
>> >
>> >
>> > int i = 4;
>> > auto L = [&j = i](auto a) {
>> >  return [k = j]() { return k; };
>> > };
>> > L(8)(); <-- ICE!
>> >
>> >
>> > All of Richard's feedback has been incorporated (I believe).
>> >
>> >
>> > Thanks!
>> >
>> > Faisal Vali
>> >
>> >
>> > _______________________________________________
>> > cfe-commits mailing list
>> > cfe-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>> >
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131207/861083f1/attachment.html>