From f5408fe484495ee4efbdd709c8a2c2fdbbbdb328 Mon Sep 17 00:00:00 2001 From: Mike Stump Date: Sat, 16 May 2009 07:57:57 +0000 Subject: Reflow some comments. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@71937 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/CodeGenModule.cpp | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) (limited to 'lib/CodeGen/CodeGenModule.cpp') diff --git a/lib/CodeGen/CodeGenModule.cpp b/lib/CodeGen/CodeGenModule.cpp index 2c966150df..be5137210f 100644 --- a/lib/CodeGen/CodeGenModule.cpp +++ b/lib/CodeGen/CodeGenModule.cpp @@ -288,9 +288,8 @@ GetLinkageForFunction(const FunctionDecl *FD, const LangOptions &Features) { /// SetFunctionDefinitionAttributes - Set attributes for a global. /// -/// FIXME: This is currently only done for aliases and functions, but -/// not for variables (these details are set in -/// EmitGlobalVarDefinition for variables). +/// FIXME: This is currently only done for aliases and functions, but not for +/// variables (these details are set in EmitGlobalVarDefinition for variables). void CodeGenModule::SetFunctionDefinitionAttributes(const FunctionDecl *D, llvm::GlobalValue *GV) { GVALinkage Linkage = GetLinkageForFunction(D, Features); @@ -703,8 +702,8 @@ llvm::Constant *CodeGenModule::GetOrCreateLLVMGlobal(const char *MangledName, // Handle things which are present even on external declarations. if (D) { - // FIXME: This code is overly simple and should be merged with - // other global handling. + // FIXME: This code is overly simple and should be merged with other global + // handling. GV->setConstant(D->getType().isConstant(Context)); // FIXME: Merge with other attribute handling code. @@ -1213,11 +1212,10 @@ GetAddrOfConstantCFString(const StringLiteral *Literal) { const llvm::Type *Ty = getTypes().ConvertType(getContext().IntTy); Ty = llvm::ArrayType::get(Ty, 0); - // FIXME: This is fairly broken if - // __CFConstantStringClassReference is already defined, in that it - // will get renamed and the user will most likely see an opaque - // error message. This is a general issue with relying on - // particular names. + // FIXME: This is fairly broken if __CFConstantStringClassReference is + // already defined, in that it will get renamed and the user will most + // likely see an opaque error message. This is a general issue with relying + // on particular names. llvm::GlobalVariable *GV = new llvm::GlobalVariable(Ty, false, llvm::GlobalVariable::ExternalLinkage, 0, @@ -1531,10 +1529,9 @@ void CodeGenModule::EmitTopLevelDecl(Decl *D) { } default: - // Make sure we handled everything we should, every other kind is - // a non-top-level decl. FIXME: Would be nice to have an - // isTopLevelDeclKind function. Need to recode Decl::Kind to do - // that easily. + // Make sure we handled everything we should, every other kind is a + // non-top-level decl. FIXME: Would be nice to have an isTopLevelDeclKind + // function. Need to recode Decl::Kind to do that easily. assert(isa(D) && "Unsupported decl kind"); } } -- cgit v1.2.3-18-g5258