Thanks, hopefully r112850 fixes it.

On Sep 2, 2010, at 11:49 AM, Jakob Stoklund Olesen wrote:

> 
> On Sep 2, 2010, at 10:49 AM, Tom Care wrote:
> 
>> Author: tcare
>> Date: Thu Sep  2 12:49:20 2010
>> New Revision: 112839
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=112839&view=rev
>> Log:
>> Improved error reporting in IdempotentOperationChecker
>> - SourceRange highlighting is only given for the relevant side of the 
>> operator (assignments give both)
>> - Added PostVisitBinaryOperator hook to retrieve the ExplodedNode for an 
>> operator
>> - Added a BugReporterVisitor to display the last store to every VarDecl in a 
>> Stmt
>> - Changed bug reporting to use the new BugReporterVisitor
>> 
>> Modified:
>>   cfe/trunk/include/clang/Checker/BugReporter/BugReporter.h
>>   cfe/trunk/lib/Checker/BugReporterVisitors.cpp
>>   cfe/trunk/lib/Checker/IdempotentOperationChecker.cpp
> 
> Hi Tom,
> 
> I think you broke the build (using gcc-4.2):
> 
> /Volumes/Data/g/clang/lib/Checker/IdempotentOperationChecker.cpp:94: error: 
> declaration of ‘<unnamed>::IdempotentOperationChecker::Assumption 
> <unnamed>::IdempotentOperationChecker::<anonymous struct>::Assumption’
> /Volumes/Data/g/clang/lib/Checker/IdempotentOperationChecker.cpp:73: error: 
> changes meaning of ‘Assumption’ from 
> ‘enum<unnamed>::IdempotentOperationChecker::Assumption’
> /Volumes/Data/g/clang/lib/Checker/IdempotentOperationChecker.cpp:95: error: 
> declaration of ‘clang::AnalysisContext* 
> <unnamed>::IdempotentOperationChecker::<anonymous struct>::AnalysisContext’
> /Volumes/Data/g/clang/lib/Checker/../../include/clang/Analysis/AnalysisContext.h:42:
>  error: changes meaning of ‘AnalysisContext’ from ‘class 
> clang::AnalysisContext’
> 
> /jakob
> 


_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to