Re: r309569 - Fix -Wshadow false positives with function-local classes.

2017-08-10 Thread Hans Wennborg via cfe-commits
r310674. Thanks! On Thu, Aug 10, 2017 at 4:52 PM, Richard Smith wrote: > Seems reasonable to me. > > On 10 August 2017 at 14:28, Hans Wennborg via cfe-commits > wrote: >> >> Should we merge this to 5.0? >> >> On Mon, Jul 31, 2017 at 8:21 AM,

Re: r309569 - Fix -Wshadow false positives with function-local classes.

2017-08-10 Thread Richard Smith via cfe-commits
Seems reasonable to me. On 10 August 2017 at 14:28, Hans Wennborg via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Should we merge this to 5.0? > > On Mon, Jul 31, 2017 at 8:21 AM, Alexander Kornienko via cfe-commits > wrote: > > Author: alexfh > > Date: Mon

Re: r309569 - Fix -Wshadow false positives with function-local classes.

2017-08-10 Thread Hans Wennborg via cfe-commits
Should we merge this to 5.0? On Mon, Jul 31, 2017 at 8:21 AM, Alexander Kornienko via cfe-commits wrote: > Author: alexfh > Date: Mon Jul 31 08:21:26 2017 > New Revision: 309569 > > URL: http://llvm.org/viewvc/llvm-project?rev=309569=rev > Log: > Fix -Wshadow false