Re: r257754 - PR25910: clang allows two var definitions with the same mangled name

2016-01-14 Thread Andrey Bokhanko via cfe-commits
Renato, Thanks! -- I committed a fix already: http://llvm.org/viewvc/llvm-project?view=revision&revision=257757 Yours, Andrey On Thu, Jan 14, 2016 at 2:57 PM, Renato Golin wrote: > On 14 January 2016 at 10:41, Andrey Bokhanko via cfe-commits > wrote: >> Author: asbokhan >> Date: Thu Jan 14 04

Re: r257754 - PR25910: clang allows two var definitions with the same mangled name

2016-01-14 Thread Renato Golin via cfe-commits
On 14 January 2016 at 10:41, Andrey Bokhanko via cfe-commits wrote: > Author: asbokhan > Date: Thu Jan 14 04:41:16 2016 > New Revision: 257754 > > URL: http://llvm.org/viewvc/llvm-project?rev=257754&view=rev > Log: > PR25910: clang allows two var definitions with the same mangled name Hi Andrey,

r257754 - PR25910: clang allows two var definitions with the same mangled name

2016-01-14 Thread Andrey Bokhanko via cfe-commits
Author: asbokhan Date: Thu Jan 14 04:41:16 2016 New Revision: 257754 URL: http://llvm.org/viewvc/llvm-project?rev=257754&view=rev Log: PR25910: clang allows two var definitions with the same mangled name Proper diagnostic and resolution of mangled names' conflicts in variables. When there is a de