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

[Argyrios Kyrtzidis]

On Feb 15, 2011, at 9:02 PM, Nick Lewycky wrote:

> 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.

Yeah but 'Checkers.inc' is put into the build directory so it won't work if build_dir != source_dir.

-Argiris

> 
> 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
>> 
>> 
>