From: Richard Trieu <rtrieu@google.com>
Date: Sat, 4 Mar 2017 02:42:41 +0000 (+0000)
Subject: Handle null QualType better in Stmt::Profile
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1b45cd564b15f4e53ebd53f0deaefc989240be51;p=clang

Handle null QualType better in Stmt::Profile

If the QualType is null, calling ASTContext::getCanonicalType on it will lead
to an assert.  This was found while testing a new use for Stmt::Profile, so
there is no test case for this.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@296956 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/AST/StmtProfile.cpp b/lib/AST/StmtProfile.cpp
index 1ac2a5bf5e..5f3a50b155 100644
--- a/lib/AST/StmtProfile.cpp
+++ b/lib/AST/StmtProfile.cpp
@@ -128,7 +128,7 @@ namespace {
     }
 
     void VisitType(QualType T) override {
-      if (Canonical)
+      if (Canonical && !T.isNull())
         T = Context.getCanonicalType(T);
 
       ID.AddPointer(T.getAsOpaquePtr());