[Lldb-commits] [lldb] r123671 - /lldb/trunk/source/Symbol/ClangASTContext.cpp

[Sean Callanan]

Author: spyffe
Date: Mon Jan 17 14:50:38 2011
New Revision: 123671

URL: http://llvm.org/viewvc/llvm-project?rev=123671&view=rev
Log:
Fixed a bug which would trigger an assert() in LLDB;
we were setting a forward-declared Objective-C class
type as being completed using an ExternalSemaSource,
but this is neither legal nor necessary.

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

Modified: lldb/trunk/source/Symbol/ClangASTContext.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Symbol/ClangASTContext.cpp?rev=123671&r1=123670&r2=123671&view=diff
==============================================================================
--- lldb/trunk/source/Symbol/ClangASTContext.cpp (original)
+++ lldb/trunk/source/Symbol/ClangASTContext.cpp Mon Jan 17 14:50:38 2011
@@ -1083,8 +1083,8 @@
             
                 if (class_interface_decl)
                 {
-                    if (has_extern)
-                        class_interface_decl->setExternallyCompleted();
+                    //if (has_extern)
+                    //    class_interface_decl->setExternallyCompleted();
                     class_interface_decl->setHasExternalLexicalStorage (has_extern);
                     //class_interface_decl->setHasExternalVisibleStorage (has_extern);
                     return true;