Re: r279374 - [Sema] Don't crash on scanf on forward-declared enums.

2016-08-21 Thread Vitaly Buka via cfe-commits
flake On Sat, Aug 20, 2016 at 11:30 AM Benjamin Kramer wrote: > It cycled back to green. flake? > > On Sat, Aug 20, 2016 at 7:27 PM, Vitaly Buka > wrote: > > msan is not happy about this change > > > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/15681/steps/check-llvm%20m

Re: r279374 - [Sema] Don't crash on scanf on forward-declared enums.

2016-08-20 Thread Benjamin Kramer via cfe-commits
It cycled back to green. flake? On Sat, Aug 20, 2016 at 7:27 PM, Vitaly Buka wrote: > msan is not happy about this change > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/15681/steps/check-llvm%20msan/logs/stdio > > On Sat, Aug 20, 2016 at 9:59 AM Benjamin Kramer via cfe-com

Re: r279374 - [Sema] Don't crash on scanf on forward-declared enums.

2016-08-20 Thread Vitaly Buka via cfe-commits
msan is not happy about this change http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/15681/steps/check-llvm%20msan/logs/stdio On Sat, Aug 20, 2016 at 9:59 AM Benjamin Kramer via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: d0k > Date: Sat Aug 20 11:51:33 2016 > N

r279374 - [Sema] Don't crash on scanf on forward-declared enums.

2016-08-20 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Aug 20 11:51:33 2016 New Revision: 279374 URL: http://llvm.org/viewvc/llvm-project?rev=279374&view=rev Log: [Sema] Don't crash on scanf on forward-declared enums. This is valid in GNU C, which allows pointers to incomplete enums. GCC just pretends that the underlying type is