Re: r284008 - Reinstate r283887 and r283882.

2016-10-12 Thread Manman via cfe-commits
> On Oct 12, 2016, at 1:39 PM, Vassil Vassilev wrote: > > Hi Manman, > On 12/10/16 20:28, Manman wrote: >> Hi Vassil, >> >> Any change between this commit and “r283887 + r283882”? > The only extra change is in: > > + bool isThisDeclarationADemotedDefinition() const { > +return isa(this) ?

Re: r284008 - Reinstate r283887 and r283882.

2016-10-12 Thread Vassil Vassilev via cfe-commits
Hi Manman, On 12/10/16 20:28, Manman wrote: Hi Vassil, Any change between this commit and “r283887 + r283882”? The only extra change is in: + bool isThisDeclarationADemotedDefinition() const { +return isa(this) ? false : + NonParmVarDeclBits.IsThisDeclarationADemotedDefinition; + }

Re: r284008 - Reinstate r283887 and r283882.

2016-10-12 Thread Manman via cfe-commits
Hi Vassil, Any change between this commit and “r283887 + r283882”? And what was the issue that caused the revert? Thanks, Manman > On Oct 12, 2016, at 4:57 AM, Vassil Vassilev via cfe-commits > wrote: > > Author: vvassilev > Date: Wed Oct 12 06:57:08 2016 > New Revision: 284008 > > URL: http