[cfe-commits] r127525 - in /cfe/trunk: lib/StaticAnalyzer/Checkers/CMakeLists.txt lib/StaticAnalyzer/Checkers/Checkers.td lib/StaticAnalyzer/Checkers/IteratorsChecker.cpp test/Analysis/iterators.cpp

[Ted Kremenek]

On Mar 30, 2011, at 7:44 PM, Chandler Carruth wrote:

> I was under the impression that the test/... directory was for regression and feature tests, not platform tests. In the past, those have typically been reserved for the LLVM test-suite, although that has no real concept or framework for testing the analyzer.
> 

Hi Chandler,

I completely agree with your arguments, but stepping back a bit w.r.t. this particular test, is this particular case actually causing a problem by including 'vector'?  It passes on all of our buildbots except win32, where we know that it doesn't work (and that is clearly documented in the test).  In lieu of the test setup that you propose (which I think is a great idea), is there any immediate action required for this test case?

Ted