commit a5f4ade268d0e5466d01c8dcb9f71fb8f3454047
author Richard Trieu <rtrieu@google.com> Sat Mar 04 02:42:41 2017 +0000
committer Richard Trieu <rtrieu@google.com> Sat Mar 04 02:42:41 2017 +0000
tree c5f39a4f06c4acbe8b50abbc8a7f3308936bb755
parent d532ddcce9076f099e30193de663ea326d6bd1c4

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.

llvm-svn: 296956

clang/lib/AST/StmtProfile.cpp

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