Thanks for the review. In r196296. -Fariborz On Dec 2, 2013, at 8:11 PM, Richard Smith <[email protected]> wrote:
> Looks good, thanks. > > On 2 Dec 2013 17:41, "jahanian" <[email protected]> wrote: > > On Dec 2, 2013, at 4:22 PM, Richard Smith <[email protected]> wrote: > >> This is a frontend bug, not an IRGen bug; the test case is ill-formed. >> "return;" can be used in a constructor or destructor, but "return >> <expression-with-type-void>;" cannot. >> > > Ok thanks. Sema patch is attached. Please review. > - Fariborz > > > > > >> >> On Mon, Dec 2, 2013 at 4:09 PM, jahanian <[email protected]> wrote: >> This patch fixes a crash when generating IRGen for armv7 which has ABI of >> implicit return of 'this' for constructors. Please review. >> // rdar://15366494 >> - fariborz >> >> >> >> >> _______________________________________________ >> 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
