Szelethus added inline comments.
================ Comment at: lib/StaticAnalyzer/Checkers/StackSizeChecker.cpp:19 +#include "clang/AST/StmtCXX.h" +#include "clang/StaticAnalyzer/Checkers/StackUsageMeasuringVisitor.h" +#include "clang/StaticAnalyzer/Core/AnalyzerOptions.h" ---------------- Szelethus wrote: > I bet you can't just `#include` this without getting some nasty buildbut > errors down the line, just `#include "StackUsageMeasuringVisitor.h"`. Oh never mind, you had it in `include/clang/StaticAnalyzer/Checkers/StackUsageMeasuringVisitor.h`. I bet it is for the clang-tidy check. Never mind. ================ Comment at: lib/StaticAnalyzer/Checkers/StackUsageMeasuringVisitor.cpp:17 + +#include <clang/StaticAnalyzer/Checkers/StackUsageMeasuringVisitor.h> + ---------------- Szelethus wrote: > #include "StackUsageMeasuringVisitor.h" Never mind, but should use `""` instead of `<>`. Repository: rC Clang https://reviews.llvm.org/D52390 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
