From 906b61f528d9cba41ed36193916675e38d1c3310 Mon Sep 17 00:00:00 2001 From: Krasimir Georgiev Date: Tue, 18 Jul 2017 07:20:53 +0000 Subject: [PATCH] [Index] Prevent canonical decl becoming nullptr Summary: This patch prevents getCanonicalDecl returning nullptr in case it finds a canonical TemplateDeclaration with no attached TemplatedDecl. Found by running the indexer over a version of the standard library deep inside a template metaprogramming mess. Reviewers: klimek, vsk Reviewed By: vsk Subscribers: vsk, arphaman, cfe-commits Differential Revision: https://reviews.llvm.org/D35212 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@308269 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Index/IndexingContext.cpp | 6 ++++-- test/Index/Core/no-templated-canonical-decl.cpp | 4 ++++ 2 files changed, 8 insertions(+), 2 deletions(-) create mode 100644 test/Index/Core/no-templated-canonical-decl.cpp diff --git a/lib/Index/IndexingContext.cpp b/lib/Index/IndexingContext.cpp index c4aa51d62f..addee691e8 100644 --- a/lib/Index/IndexingContext.cpp +++ b/lib/Index/IndexingContext.cpp @@ -260,8 +260,10 @@ static const Decl *adjustParent(const Decl *Parent) { static const Decl *getCanonicalDecl(const Decl *D) { D = D->getCanonicalDecl(); if (auto TD = dyn_cast(D)) { - D = TD->getTemplatedDecl(); - assert(D->isCanonicalDecl()); + if (auto TTD = TD->getTemplatedDecl()) { + D = TTD; + assert(D->isCanonicalDecl()); + } } return D; diff --git a/test/Index/Core/no-templated-canonical-decl.cpp b/test/Index/Core/no-templated-canonical-decl.cpp new file mode 100644 index 0000000000..5434e9cb65 --- /dev/null +++ b/test/Index/Core/no-templated-canonical-decl.cpp @@ -0,0 +1,4 @@ +// RUN: c-index-test core -print-source-symbols -include-locals -- %s | FileCheck %s + +template