[Lldb-commits] [lldb] r247196 - Now that CompilerType uses a "TypeSystem *" and a "void *" instead of a "clang::ASTContext *" and a "void *", we need to know if anyone is trying to create a CompilerType from a clang::ASTContext that has no backing ClangASTContext.

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 9 15:13:01 PDT 2015


Author: gclayton
Date: Wed Sep  9 17:13:01 2015
New Revision: 247196

URL: http://llvm.org/viewvc/llvm-project?rev=247196&view=rev
Log:
Now that CompilerType uses a "TypeSystem *" and a "void *" instead of a "clang::ASTContext *" and a "void *", we need to know if anyone is trying to create a CompilerType from a clang::ASTContext that has no backing ClangASTContext.

This assert will fire if this is the case and we will need to fix the offending code.


Modified:
    lldb/trunk/source/Symbol/CompilerType.cpp

Modified: lldb/trunk/source/Symbol/CompilerType.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Symbol/CompilerType.cpp?rev=247196&r1=247195&r2=247196&view=diff
==============================================================================
--- lldb/trunk/source/Symbol/CompilerType.cpp (original)
+++ lldb/trunk/source/Symbol/CompilerType.cpp Wed Sep  9 17:13:01 2015
@@ -42,6 +42,10 @@ CompilerType::CompilerType (clang::ASTCo
     m_type (qual_type.getAsOpaquePtr()),
     m_type_system (ClangASTContext::GetASTContext(ast))
 {
+#ifdef LLDB_CONFIGURATION_DEBUG
+    if (m_type)
+        assert(m_type_system != nullptr);
+#endif
 }
 
 CompilerType::~CompilerType()




More information about the lldb-commits mailing list