[cfe-commits] r80642 - in /cfe/trunk: include/clang/AST/Expr.h include/clang/AST/ExprCXX.h include/clang/AST/StmtNodes.def lib/AST/Expr.cpp lib/AST/StmtPrinter.cpp lib/AST/StmtProfile.cpp lib/Analysis/GRExprEngine.cpp lib/CodeGen/CGExpr.cpp lib/Sema/SemaChecking.cpp lib/Sema/SemaExpr.cpp lib/Sema/TreeTransform.h

[Douglas Gregor]

On Aug 31, 2009, at 5:16 PM, Mike Stump wrote:

> On Aug 31, 2009, at 4:41 PM, Douglas Gregor wrote:
>> Author: dgregor
>> Date: Mon Aug 31 18:41:50 2009
>> New Revision: 80642
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=80642&view=rev
>> Log:
>> Eliminate CXXAdornedMemberExpr entirely. Instead, optionally allocate
>> space within the MemberExpr for the nested-name-specifier and its
>> source range. We'll do the same thing with explicitly-specified
>> template arguments, assuming I don't flip-flop again.
>
> In file included from SemaTemplate.cpp:13:
> TreeTransform.h: In member function  
> ‘clang::ASTOwningResult<&clang::ActionBase::DeleteExpr>  
> clang::TreeTransform<Derived>::TransformMemberExpr 
> (clang::MemberExpr*)’:
> TreeTransform.h:2967: warning: empty body in an if-statement
> In file included from SemaTemplateInstantiate.cpp:14:
> TreeTransform.h: In member function  
> ‘clang::ASTOwningResult<&clang::ActionBase::DeleteExpr>  
> clang::TreeTransform<Derived>::TransformMemberExpr 
> (clang::MemberExpr*)’:
> TreeTransform.h:2967: warning: empty body in an if-statement
>
> A warning is a terrible thing to waste.  :-)

Ho ho! Thanks for pointing this out; it didn't show up for me in the  
Xcode build (???).

	- Doug