Re: r288089 - Avoid lambdas in default member initializers to work around clang bug

2016-12-06 Thread Benjamin Kramer via cfe-commits
Leaving it as it is now is fine with me. On Tue, Dec 6, 2016 at 8:38 PM, Reid Kleckner wrote: > This can be reverted now that PR31197 is fixed. I kind of like the new code > better, so up to you, Benjamin. > > On Mon, Nov 28, 2016 at 3:58 PM, Reid Kleckner via cfe-commits > wrote: >> >> Author:

Re: r288089 - Avoid lambdas in default member initializers to work around clang bug

2016-12-06 Thread Reid Kleckner via cfe-commits
This can be reverted now that PR31197 is fixed. I kind of like the new code better, so up to you, Benjamin. On Mon, Nov 28, 2016 at 3:58 PM, Reid Kleckner via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rnk > Date: Mon Nov 28 17:58:04 2016 > New Revision: 288089 > > URL: http://llv

r288089 - Avoid lambdas in default member initializers to work around clang bug

2016-11-28 Thread Reid Kleckner via cfe-commits
Author: rnk Date: Mon Nov 28 17:58:04 2016 New Revision: 288089 URL: http://llvm.org/viewvc/llvm-project?rev=288089&view=rev Log: Avoid lambdas in default member initializers to work around clang bug On Windows, Clang is mangling lambdas in default member initializers incorrectly. See PR31197. T