From f4ffa0dabfb6cb9c6607a8954f99719b86e856f4 Mon Sep 17 00:00:00 2001 From: Daniel Dunbar Date: Tue, 30 Mar 2010 22:26:07 +0000 Subject: [PATCH] Minor formatting/FIXME cleanups. llvm-svn: 99944 --- clang/lib/CodeGen/CGExpr.cpp | 1 - clang/lib/CodeGen/CodeGenTypes.cpp | 2 +- clang/lib/CodeGen/CodeGenTypes.h | 4 +--- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/clang/lib/CodeGen/CGExpr.cpp b/clang/lib/CodeGen/CGExpr.cpp index dc9ecd64f4e4..2ddc08a4c449 100644 --- a/clang/lib/CodeGen/CGExpr.cpp +++ b/clang/lib/CodeGen/CGExpr.cpp @@ -1872,7 +1872,6 @@ LValue CodeGenFunction::EmitObjCSuperExprLValue(const ObjCSuperExpr *E) { LValue CodeGenFunction::EmitStmtExprLValue(const StmtExpr *E) { // Can only get l-value for message expression returning aggregate type RValue RV = EmitAnyExprToTemp(E); - // FIXME: can this be volatile? return LValue::MakeAddr(RV.getAggregateAddr(), MakeQualifiers(E->getType())); } diff --git a/clang/lib/CodeGen/CodeGenTypes.cpp b/clang/lib/CodeGen/CodeGenTypes.cpp index 4feca4dd7d3d..4f37537769f9 100644 --- a/clang/lib/CodeGen/CodeGenTypes.cpp +++ b/clang/lib/CodeGen/CodeGenTypes.cpp @@ -477,7 +477,7 @@ void CodeGenTypes::addFieldInfo(const FieldDecl *FD, unsigned No) { FieldInfo[FD] = No; } -/// getBitFieldInfo - Return the BitFieldInfo that corresponds to the field FD. +/// getBitFieldInfo - Return the BitFieldInfo that corresponds to the field FD. CodeGenTypes::BitFieldInfo CodeGenTypes::getBitFieldInfo(const FieldDecl *FD) { llvm::DenseMap::iterator I = BitFields.find(FD); diff --git a/clang/lib/CodeGen/CodeGenTypes.h b/clang/lib/CodeGen/CodeGenTypes.h index 173c4ad37814..9a620e4c0122 100644 --- a/clang/lib/CodeGen/CodeGenTypes.h +++ b/clang/lib/CodeGen/CodeGenTypes.h @@ -69,7 +69,7 @@ namespace CodeGen { public: CGRecordLayout(const llvm::Type *T, bool ContainsPointerToDataMember) - : LLVMType(T), ContainsPointerToDataMember(ContainsPointerToDataMember) { } + : LLVMType(T), ContainsPointerToDataMember(ContainsPointerToDataMember) {} /// getLLVMType - Return llvm type associated with this record. const llvm::Type *getLLVMType() const { @@ -107,8 +107,6 @@ class CodeGenTypes { /// CGRecordLayouts - This maps llvm struct type with corresponding /// record layout info. - /// FIXME : If CGRecordLayout is less than 16 bytes then use - /// inline it in the map. llvm::DenseMap CGRecordLayouts; /// FieldInfo - This maps struct field with corresponding llvm struct type