Re: r311182 - [analyzer] Fix modeling of constructors

2017-08-21 Thread Alexander Shaposhnikov via cfe-commits
Thanks! On Mon, Aug 21, 2017 at 1:28 PM, Hans Wennborg via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Merged in r311378. > > Thanks, > Hans > > On Mon, Aug 21, 2017 at 1:12 PM, Anna Zaks wrote: > > I approve.Thanks Hans! > > Anna > >> On Aug 21, 2017, at 1:05 PM, Hans Wennborg wrote: >

Re: r311182 - [analyzer] Fix modeling of constructors

2017-08-21 Thread Hans Wennborg via cfe-commits
Merged in r311378. Thanks, Hans On Mon, Aug 21, 2017 at 1:12 PM, Anna Zaks wrote: > I approve.Thanks Hans! > Anna >> On Aug 21, 2017, at 1:05 PM, Hans Wennborg wrote: >> >> I'm ok with it if Anna approves. >> >> On Mon, Aug 21, 2017 at 9:06 AM, Artem Dergachev wrote: >>> Hello, >>> >>> Do we h

Re: r311182 - [analyzer] Fix modeling of constructors

2017-08-21 Thread Anna Zaks via cfe-commits
I approve.Thanks Hans! Anna > On Aug 21, 2017, at 1:05 PM, Hans Wennborg wrote: > > I'm ok with it if Anna approves. > > On Mon, Aug 21, 2017 at 9:06 AM, Artem Dergachev wrote: >> Hello, >> >> Do we have time to merge this change into release 5.0.0? It's an assertion >> failure fix, which show

Re: r311182 - [analyzer] Fix modeling of constructors

2017-08-21 Thread Hans Wennborg via cfe-commits
I'm ok with it if Anna approves. On Mon, Aug 21, 2017 at 9:06 AM, Artem Dergachev wrote: > Hello, > > Do we have time to merge this change into release 5.0.0? It's an assertion > failure fix, which shows up on C++ code involving double-inheritance with > empty base classes. > > Artem. > > > On 8/

Re: r311182 - [analyzer] Fix modeling of constructors

2017-08-21 Thread Artem Dergachev via cfe-commits
Hello, Do we have time to merge this change into release 5.0.0? It's an assertion failure fix, which shows up on C++ code involving double-inheritance with empty base classes. Artem. On 8/18/17 9:20 PM, Alexander Shaposhnikov via cfe-commits wrote: Author: alexshap Date: Fri Aug 18 11:20:43