Author: shafik Date: 2020-07-24T13:15:08-07:00 New Revision: 0db2934b0fa9e00ac98e2cb168adba96f6bcd0da
URL: https://github.com/llvm/llvm-project/commit/0db2934b0fa9e00ac98e2cb168adba96f6bcd0da DIFF: https://github.com/llvm/llvm-project/commit/0db2934b0fa9e00ac98e2cb168adba96f6bcd0da.diff LOG: [ASTImporter] Modify ImportDefiniton for ObjCInterfaceDecl so that we always the ImportDeclContext one we start the definition Once we start the definition of an ObjCInterfaceDecl we won't attempt to ImportDeclContext later on. Unlike RecordDecl case which uses DefinitionCompleter to force completeDefinition we don't seem to have a similar mechanism for ObjCInterfaceDecl. This fix was needed due to a bug we see in LLDB expression parsing where an initial expression cause an ObjCInterfaceDecl to be defined and subsequent expressions during import do not call ImportDeclContext and we can end up in a situation where ivars are imported out of order and not all ivars are imported. Differential Revision: https://reviews.llvm.org/D83972 Added: Modified: clang/lib/AST/ASTImporter.cpp lldb/test/API/lang/objc/bitfield_ivars/TestBitfieldIvars.py Removed: ################################################################################ diff --git a/clang/lib/AST/ASTImporter.cpp b/clang/lib/AST/ASTImporter.cpp index fcfaba625a722..e0bca8f08bb41 100644 --- a/clang/lib/AST/ASTImporter.cpp +++ b/clang/lib/AST/ASTImporter.cpp @@ -4758,11 +4758,10 @@ Error ASTNodeImporter::ImportDefinition( return ToImplOrErr.takeError(); } - if (shouldForceImportDeclContext(Kind)) { - // Import all of the members of this class. - if (Error Err = ImportDeclContext(From, /*ForceImport=*/true)) - return Err; - } + // Import all of the members of this class. + if (Error Err = ImportDeclContext(From, /*ForceImport=*/true)) + return Err; + return Error::success(); } diff --git a/lldb/test/API/lang/objc/bitfield_ivars/TestBitfieldIvars.py b/lldb/test/API/lang/objc/bitfield_ivars/TestBitfieldIvars.py index 6118854131024..4154bb144b350 100644 --- a/lldb/test/API/lang/objc/bitfield_ivars/TestBitfieldIvars.py +++ b/lldb/test/API/lang/objc/bitfield_ivars/TestBitfieldIvars.py @@ -14,9 +14,8 @@ def test(self): lldbutil.run_to_source_breakpoint(self, "// break here", lldb.SBFileSpec("main.m")) self.expect_expr("chb->hb->field1", result_type="unsigned int", result_value="0") - - ## FIXME field2 should have a value of 1 - self.expect("expr chb->hb->field2", matching=False, substrs = ["= 1"]) # this must happen second + ## This should happen second + self.expect_expr("chb->hb->field2", result_type="unsigned int", result_value="1") self.expect_expr("hb2->field1", result_type="unsigned int", result_value="10") self.expect_expr("hb2->field2", result_type="unsigned int", result_value="3") _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits