[cfe-dev] lifetime of conditional temporaries

[Anders Carlsson]

Hi Victor,

the patches look great! Thanks for fixing this :)

One small comment - could you move the conditional temporaries tests into a separate file? You could call it conditional-temporaries.cpp

Thanks,
Anders

17 nov 2009 kl. 23.20 skrev Victor Zverovich:

> The first patch (cond-temp-test.patch) adds 3 new test cases that check the lifetime of conditional temporaries.
> The second patch (cond-temp.patch) fixes the bug making the new test cases pass.
> 
> Victor
> 
> 2009/11/17 Victor Zverovich <victor.zverovich at googlemail.com>
> I submitted it here http://llvm.org/bugs/show_bug.cgi?id=5539 so that it is not lost until I come up with a fix.
> 
> Victor
> 
> 2009/11/17 Anders Carlsson <andersca at mac.com>
> 
> 17 nov 2009 kl. 04.44 skrev Victor Zverovich:
> 
> > Dear all
> >
> > The same applies to && and || operators. Is this intended or needs to be fixed in clang?
> > In the latter case and if nobody is already working on it I can try fixing it.
> >
> 
> Hi Victor,
> 
> this should work. Please file a bug.
> 
> Anders
> 
> > Best regards,
> > Victor
> >
> > _______________________________________________
> > cfe-dev mailing list
> > cfe-dev at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
> 
> 
> 
> <cond-temp-test.patch><cond-temp.patch>_______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20091118/67f76c4c/attachment.html>