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

Bill Wendling isanbard at gmail.com
Sat Dec 7 15:52:07 PST 2013


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/1a205dcf/attachment.html>


More information about the cfe-commits mailing list