Re: r325321 - [Sema] Take into account the current context when checking the

2018-02-16 Thread Akira Hatanaka via cfe-commits
Thanks for taking care of it. It looks like I had to traverse the chain of Scopes upwards until Scope::getEntity() returns something that is not null. This is a reduced test case. clang doesn’t crash if I remove “if (true)”. $ cat test1.cpp struct CmapSubtable {}; struct A { protected: typ

Re: r325321 - [Sema] Take into account the current context when checking the

2018-02-16 Thread Hans Wennborg via cfe-commits
Reverted in r325335 as this broke the Chromium build. See https://bugs.chromium.org/p/chromium/issues/detail?id=813017 for stack trace and reproducer. (I think Ben said it might have also broken a Clang bootstrap build, but I didn't see that anywhere?) On Fri, Feb 16, 2018 at 9:47 AM, Akira Hata