[libcxx] r216317 - Add return statement to a test class's assignment operator. Defect found by Coverity Scan.
Justin Bogner
mail at justinbogner.com
Tue Aug 26 18:57:22 PDT 2014
Eric Fiselier <eric at efcs.ca> writes:
> That looks doable. I'll look into it. Thanks.
> Can you point me to some documentation on `-verify`?
This should help:
http://clang.llvm.org/doxygen/classclang_1_1VerifyDiagnosticConsumer.html#details
> /Eric
>
> On Tue, Aug 26, 2014 at 7:36 PM, Justin Bogner <mail at justinbogner.com> wrote:
>
> Eric Fiselier <eric at efcs.ca> writes:
> > I'm not opposed to the change but using Filecheck really hurts our
> ability to
> > test standalone.
> > Also, we don't have that many compilation failures tests. I'm not sure
> how
> > badly this change is needed.
>
> Using clang -verify avoids that issue, but there's still the matter of
> having to update almost 300 tests to use it.
>
> Here's a quick example of hackily adding a verify flag instead of
> checking for rc 1 in the lit.cfg and updating the test under discussion:
More information about the cfe-commits
mailing list