More PushOnScopeChain() FIXME's.

llvm-svn: 69894

clang/lib/Frontend/PCHReader.cpp

@@ -2743,7 +2743,6 @@ PCHReader::ReadDeclarationName(const RecordData &Record, unsigned &Idx) {
   case DeclarationName::ObjCOneArgSelector:
   case DeclarationName::ObjCMultiArgSelector:
     return DeclarationName(GetSelector(Record, Idx));
-    break;
 
   case DeclarationName::CXXConstructorName:
     return Context.DeclarationNames.getCXXConstructorName(

clang/lib/Sema/SemaDeclObjC.cpp

@@ -206,8 +206,7 @@ Sema::DeclPtrTy Sema::ActOnCompatiblityAlias(SourceLocation AtLoc,
   
   ObjCAliasDecls[AliasName] = AliasDecl;
 
-  // FIXME: PushOnScopeChains?
-  CurContext->addDecl(Context, AliasDecl);
+  PushOnScopeChains(AliasDecl, TUScope);
   if (!CheckObjCDeclScope(AliasDecl))
     TUScope->AddDecl(DeclPtrTy::make(AliasDecl));
 
@@ -1048,8 +1047,7 @@ Sema::ActOnForwardClassDeclaration(SourceLocation AtClassLoc,
                                         IdentList[i], SourceLocation(), true);
       ObjCInterfaceDecls[IdentList[i]] = IDecl;
 
-      // FIXME: PushOnScopeChains?
-      CurContext->addDecl(Context, IDecl);
+      PushOnScopeChains(IDecl, TUScope);
       // Remember that this needs to be removed when the scope is popped.
       TUScope->AddDecl(DeclPtrTy::make(IDecl));
     }

clang/test/PCH/objc_methods.h

@@ -4,3 +4,8 @@
 + alloc;
 - (void)instMethod;
 @end
+
+@class TestForwardClassDecl;
+
+// FIXME: @compatibility_alias  AliasForTestPCH TestPCH;
+

clang/test/PCH/objc_methods.m

@@ -7,6 +7,10 @@
 
 void func() {
  TestPCH *xx;
+ TestForwardClassDecl *yy;
+// FIXME:
+// AliasForTestPCH *zz;
+ 
 #if 0
  xx = [TestPCH alloc];
  [xx instMethod];