]> granicus.if.org Git - clang/commit
[ASTImporter] Fix isa cast assert
authorPeter Szecsi <szepet95@gmail.com>
Wed, 2 May 2018 11:52:54 +0000 (11:52 +0000)
committerPeter Szecsi <szepet95@gmail.com>
Wed, 2 May 2018 11:52:54 +0000 (11:52 +0000)
commitaeaf7d71984a08bf719f386880f67fd410408d03
tree87605e456f3833a05536445bfa123b2de3a6bd66
parenta05f37359b23be7c068e19968c8f106edf6f2b34
[ASTImporter] Fix isa cast assert

Do early return if we can't import the found decl for a member expr.
This follows the pre-existing scheme, e.g with E->getMemberDecl().
E->getFoundDecl().getDecl() can be null when a member expression does
not involve lookup. It may involve a lookup in case of a using directive
which refers to a member function in a base class template.

We faced this assert during the CTU analysis of google::protobuf v3.5.2.
We tried hard to synthesize a minimal test example both by hand and by
executing creduce on multiple files. Unfortunately, we were unable to reduce
to such a minimal example, yet. Nevertheless, this fix solved the problem in
protobuf.

To reproduce the error one must execute the analyzer with
-Xclang -analyzer-config -Xclang experimental-enable-naive-ctu-analysis=true -Xclang -analyzer-config -Xclang ctu-dir=/path/to/ctu_dir

Patch by Gabor Marton!

Differential Revision: https://reviews.llvm.org/D46019

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@331344 91177308-0d34-0410-b5e6-96231b3b80d8
lib/AST/ASTImporter.cpp