[cfe-commits] r125645 - /cfe/trunk/lib/StaticAnalyzer/Checkers/ClangSACheckers.h

Nick Lewycky nicholas at mxc.ca
Tue Feb 15 21:02:04 PST 2011


Argyrios Kyrtzidis wrote:
> On Feb 15, 2011, at 6:34 PM, Nick Lewycky wrote:
>
>> Author: nicholas
>> Date: Tue Feb 15 20:34:28 2011
>> New Revision: 125645
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=125645&view=rev
>> Log:
>> Revert r125642. This broke the build? It should be a no-op.
>
> "../Checkers/Checkers.inc" is necessary because "ClangSACheckers.h" is also included by lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp.

No.

If you create:
   test/a/1.h  with "TEST DATA"
   test/a/2.h  with "#include "1.h"
   test/b/1.c  with "#include "../a/2.h"
and from test/b run "$CC -E 1.c" then it will indeed include a/1.h.

Nick

> -Argiris
>
>>
>> Modified:
>>     cfe/trunk/lib/StaticAnalyzer/Checkers/ClangSACheckers.h
>>
>> Modified: cfe/trunk/lib/StaticAnalyzer/Checkers/ClangSACheckers.h
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/StaticAnalyzer/Checkers/ClangSACheckers.h?rev=125645&r1=125644&r2=125645&view=diff
>> ==============================================================================
>> --- cfe/trunk/lib/StaticAnalyzer/Checkers/ClangSACheckers.h (original)
>> +++ cfe/trunk/lib/StaticAnalyzer/Checkers/ClangSACheckers.h Tue Feb 15 20:34:28 2011
>> @@ -23,7 +23,7 @@
>> #define GET_CHECKERS
>> #define CHECKER(FULLNAME,CLASS,CXXFILE,HELPTEXT,HIDDEN)    \
>>    void register##CLASS(ExprEngine&Eng);
>> -#include "Checkers.inc"
>> +#include "../Checkers/Checkers.inc"
>> #undef CHECKER
>> #undef GET_CHECKERS
>>
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>




More information about the cfe-commits mailing list