diff options
| -rw-r--r-- | clang/lib/Rewrite/RewriteModernObjC.cpp | 24 | ||||
| -rw-r--r-- | clang/test/Rewriter/objc-modern-linkage-spec.mm | 21 | 
2 files changed, 45 insertions, 0 deletions
| diff --git a/clang/lib/Rewrite/RewriteModernObjC.cpp b/clang/lib/Rewrite/RewriteModernObjC.cpp index 91e89d17590..45369a95dea 100644 --- a/clang/lib/Rewrite/RewriteModernObjC.cpp +++ b/clang/lib/Rewrite/RewriteModernObjC.cpp @@ -329,6 +329,7 @@ namespace {      Stmt *RewriteBreakStmt(BreakStmt *S);      Stmt *RewriteContinueStmt(ContinueStmt *S);      void RewriteCastExpr(CStyleCastExpr *CE); +    void RewriteLinkageSpec(LinkageSpecDecl *LSD);      // Block rewriting.      void RewriteBlocksInFunctionProtoType(QualType funcType, NamedDecl *D); @@ -672,6 +673,7 @@ void RewriteModernObjC::HandleTopLevelSingleDecl(Decl *D) {      if (PD->isThisDeclarationADefinition())        RewriteProtocolDecl(PD);    } else if (LinkageSpecDecl *LSD = dyn_cast<LinkageSpecDecl>(D)) { +    RewriteLinkageSpec(LSD);      // Recurse into linkage specifications      for (DeclContext::decl_iterator DI = LSD->decls_begin(),                                   DIEnd = LSD->decls_end(); @@ -693,6 +695,12 @@ void RewriteModernObjC::HandleTopLevelSingleDecl(Decl *D) {            RewriteForwardClassDecl(DG);            continue;          } +        else { +          // Keep track of all interface declarations seen. +          ObjCInterfacesSeen.push_back(IFace); +          ++DI; +          continue; +        }        }        if (ObjCProtocolDecl *Proto = dyn_cast<ObjCProtocolDecl>((*DI))) { @@ -1058,6 +1066,22 @@ RewriteModernObjC::RewriteForwardProtocolDecl(const llvm::SmallVector<Decl*, 8>    ReplaceText(LocStart, 0, "// ");  } +void  +RewriteModernObjC::RewriteLinkageSpec(LinkageSpecDecl *LSD) { +  SourceLocation LocStart = LSD->getExternLoc(); +  if (LocStart.isInvalid()) +    llvm_unreachable("Invalid extern SourceLocation"); +   +  ReplaceText(LocStart, 0, "// "); +  if (!LSD->hasBraces()) +    return; +  // FIXME. We don't rewrite well if '{' is not on same line as 'extern'. +  SourceLocation LocRBrace = LSD->getRBraceLoc(); +  if (LocRBrace.isInvalid()) +    llvm_unreachable("Invalid rbrace SourceLocation"); +  ReplaceText(LocRBrace, 0, "// "); +} +  void RewriteModernObjC::RewriteTypeIntoString(QualType T, std::string &ResultStr,                                          const FunctionType *&FPRetType) {    if (T->isObjCQualifiedIdType()) diff --git a/clang/test/Rewriter/objc-modern-linkage-spec.mm b/clang/test/Rewriter/objc-modern-linkage-spec.mm new file mode 100644 index 00000000000..028d78702fd --- /dev/null +++ b/clang/test/Rewriter/objc-modern-linkage-spec.mm @@ -0,0 +1,21 @@ +// RUN: %clang_cc1 -x objective-c++ -fms-extensions -rewrite-objc %s -o %t-rw.cpp +// RUN: %clang_cc1 -fsyntax-only -Wno-attributes -D"__declspec(X)=" %t-rw.cpp +// rdar://11169733 + +extern "C" __declspec(dllexport) +@interface Test @end + +@implementation Test @end + +extern "C" { +__declspec(dllexport) +@interface Test1 @end + +@implementation Test1 @end + +__declspec(dllexport) +@interface Test2 @end + +@implementation Test2 @end +}; + | 

