Re: r277787 - [Sema] Add sizeof diagnostics for bzero

2016-08-10 Thread Bruno Cardoso Lopes via cfe-commits
This typo was exactly the reason why, r278264. Thanks On Fri, Aug 5, 2016 at 10:09 AM, David Majnemer wrote: > > > On Thu, Aug 4, 2016 at 4:55 PM, Bruno Cardoso Lopes via cfe-commits > wrote: >> >> Author: bruno >> Date: Thu Aug 4 18:55:22 2016 >> New Revision: 277787 >> >> URL: http://llvm.org

Re: r277787 - [Sema] Add sizeof diagnostics for bzero

2016-08-05 Thread David Majnemer via cfe-commits
On Thu, Aug 4, 2016 at 4:55 PM, Bruno Cardoso Lopes via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: bruno > Date: Thu Aug 4 18:55:22 2016 > New Revision: 277787 > > URL: http://llvm.org/viewvc/llvm-project?rev=277787&view=rev > Log: > [Sema] Add sizeof diagnostics for bzero > > For

Re: r277787 - [Sema] Add sizeof diagnostics for bzero

2016-08-05 Thread Bruno Cardoso Lopes via cfe-commits
Hi Gabor, Thanks for the PR, I reverted it in r277830 until I can manage to fix it. On Fri, Aug 5, 2016 at 6:44 AM, Gabor Ballabas wrote: > Hi Bruno, > > My name is Gabor Ballabas, I work at the University of Szeged, Hungary. > We run daily code size benchmarks testing trunk Clang. After your pa

Re: r277787 - [Sema] Add sizeof diagnostics for bzero

2016-08-05 Thread Gabor Ballabas via cfe-commits
Hi Bruno, My name is Gabor Ballabas, I work at the University of Szeged, Hungary. We run daily code size benchmarks testing trunk Clang. After your patch landed we started to get a Clang crash when compiling one of the files in our benchmark system. I created a bugreport with the details: htt