[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