If the test fails without it, then yes.
On Wed, Nov 27, 2013 at 4:55 PM, David Majnemer <[email protected]>wrote: > Do we want this merged into 3.4? > > Sent from my iPhone > > > On Nov 27, 2013, at 7:13 PM, Kaelyn Uhrain <[email protected]> wrote: > > > > Author: rikka > > Date: Wed Nov 27 18:13:38 2013 > > New Revision: 195897 > > > > URL: http://llvm.org/viewvc/llvm-project?rev=195897&view=rev > > Log: > > Add a return statement to the func with an int return type. > > > > Modified: > > cfe/trunk/test/SemaTemplate/instantiate-local-class.cpp > > > > Modified: cfe/trunk/test/SemaTemplate/instantiate-local-class.cpp > > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaTemplate/instantiate-local-class.cpp?rev=195897&r1=195896&r2=195897&view=diff > > > ============================================================================== > > --- cfe/trunk/test/SemaTemplate/instantiate-local-class.cpp (original) > > +++ cfe/trunk/test/SemaTemplate/instantiate-local-class.cpp Wed Nov 27 > 18:13:38 2013 > > @@ -178,5 +178,6 @@ namespace PR14373 { > > }; > > int call() { > > exec_func(Type()); > > + return 0; > > } > > } > > > > > > _______________________________________________ > > cfe-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
