[clang-tools-extra] r245471 - [clang-tidy] Fix a bug in UseNullptrCheck.
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 19 18:36:57 PDT 2015
The tests are passing now
<http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA_check/6531/>.
Will now try to add back the piece of test that was removed and see whether
it works.
On Thu, Aug 20, 2015 at 2:00 AM, Alexander Kornienko <alexfh at google.com>
wrote:
> After looking at the code once again, I found an obvious use-after-free
> bug which could have caused all of this (and for some reason only
> manifested itself on Darwin). r245524 should fix the issue.
>
>
> On Thu, Aug 20, 2015 at 1:44 AM, Justin Bogner <mail at justinbogner.com>
> wrote:
>
>> That didn't work either - both tests are still failing.
>>
>> Alexander Kornienko <alexfh at google.com> writes:
>> > Something weird happens with options reading. Submitted a possible
>> workaround
>> > in r245517.
>> >
>> > On Thu, Aug 20, 2015 at 12:48 AM, Justin Bogner <mail at justinbogner.com>
>> wrote:
>> >
>> > Locally, it seems to fail two tests now:
>> >
>> > Failing Tests (2):
>> > Clang Tools :: clang-tidy/modernize-use-nullptr-basic.cpp
>> > Clang Tools :: clang-tidy/modernize-use-nullptr.cpp
>> >
>> > I'll poke at it for a minute and see if there's something obvious I
>> can
>> > do, but running the run line manually just seems to return 127 with
>> no
>> > output for me (as opposed to the output that's printed correctly
>> under
>> > lit)
>> >
>> > Alexander Kornienko via cfe-commits <cfe-commits at lists.llvm.org>
>> writes:
>> > > I've committed the check with minor modifications and without the
>> > offending
>> > > test in r245511. Could you verify that it works in your setup?
>> > >
>> > > -- Alex
>> > >
>> > > On Wed, Aug 19, 2015 at 11:41 PM, Pete Cooper <
>> peter_cooper at apple.com>
>> > wrote:
>> > >
>> > > On Aug 19, 2015, at 2:38 PM, Alexander Kornienko <
>> > alexfh at google.com>
>> > > wrote:
>> > >
>> > > The check has been reverted in r245493. Sorry for the
>> breakage,
>> > I was
>> > > hoping that this commit fixes it.
>> > >
>> > > No problem. Thanks for taking a look.
>> > >
>> > > If you think the rest of the code is good and want to land it
>> again
>> > then
>> > > thats fine. You can leave out the check below until you are
>> happy
>> > that
>> > > its working too. That way you potentially won’t be blocked
>> for too
>> > long.
>> > >
>> > > Thanks,
>> > > Pete
>> > >
>> > > On Wed, Aug 19, 2015 at 10:31 PM, Pete Cooper <
>> > peter_cooper at apple.com>
>> > > wrote:
>> > >
>> > > Looks like its only a single test thats failing.
>> > >
>> > > Would you mind if I remove this piece of the test
>> until we
>> > can get
>> > > to the bottom of it?
>> > >
>> > > void test_macro_expansion4() {
>> > > #define MY_NULL NULL
>> > > int *p = MY_NULL;
>> > > // CHECK-MESSAGES: :[[@LINE-1]]:12: warning: use
>> > nullptr
>> > > [modernize-use-nullptr]
>> > > // CHECK-FIXES: int *p = nullptr;
>> > > #undef MY_NULL
>> > > }
>> > >
>> > > Thanks,
>> > > Pete
>> > >
>> > > On Aug 19, 2015, at 1:00 PM, Pete Cooper <
>> > > peter_cooper at apple.com> wrote:
>> > >
>> > > Hi Alexander
>> > >
>> > > We’re still getting a green dragon failure on the
>> null
>> > ptr
>> > > check test. Mind taking a look?
>> > >
>> > > http://lab.llvm.org:8080/green/job/
>> > > clang-stage1-configure-RA_check/10351/consoleFull#
>> > > 50560140149ba4694-19c4-4d7e-bec5-911270d8a58c
>> > >
>> > > Thanks
>> > > Pete
>> > >
>> > > On Aug 19, 2015, at 10:50 AM, Alexander
>> Kornienko
>> > via
>> > > cfe-commits <cfe-commits at lists.llvm.org>
>> wrote:
>> > >
>> > > Author: alexfh
>> > > Date: Wed Aug 19 12:50:22 2015
>> > > New Revision: 245471
>> > >
>> > > URL:
>> https://urldefense.proofpoint.com/v2/url?u=
>> > >
>> > http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D245471-26view-3Drev
>> > > &d=BQIGaQ&c=eEvniauFctOgLOKGJOplqw&r=
>> > > 03tkj3107244TlY4t3_hEgkDY-UG6gKwwK0wOUS3qjM&m=
>> > > vEGzlOUc6IO5ny5JKNkJAUEoiokQ1N60GDcHk0yboKQ&s=
>> > > wuoYp-wW8aBSkIHSX7igi7DHfur7JyIHWwnzHMTYdlg&e=
>> > > Log:
>> > > [clang-tidy] Fix a bug in UseNullptrCheck.
>> > >
>> > > https://urldefense.proofpoint.com/v2/url?u=
>> > > http-3A__reviews.llvm.org_D12162&d=BQIGaQ&c=
>> > > eEvniauFctOgLOKGJOplqw&r=
>> > > 03tkj3107244TlY4t3_hEgkDY-UG6gKwwK0wOUS3qjM&m=
>> > > vEGzlOUc6IO5ny5JKNkJAUEoiokQ1N60GDcHk0yboKQ&s=
>> > > YiXUYCqfOl7durvaPOdifn3l7_G0FJhlE4A_q5Q6xwM&e=
>> > >
>> > > Patch by Angel Garcia!
>> > >
>> > > Modified:
>> > >
>> clang-tools-extra/trunk/clang-tidy/modernize/
>> > > UseNullptrCheck.cpp
>> > >
>> > > Modified: clang-tools-extra/trunk/clang-tidy/
>> > modernize/
>> > > UseNullptrCheck.cpp
>> > > URL:
>> https://urldefense.proofpoint.com/v2/url?u=
>> > >
>> >
>> http-3A__llvm.org_viewvc_llvm-2Dproject_clang-2Dtools-2Dextra_trunk_clang-2Dtidy_modernize_UseNullptrCheck.cpp-3Frev-3D245471-26r1-3D245470-26r2-3D245471-26view-3Ddiff
>> > > &d=BQIGaQ&c=eEvniauFctOgLOKGJOplqw&r=
>> > > 03tkj3107244TlY4t3_hEgkDY-UG6gKwwK0wOUS3qjM&m=
>> > > vEGzlOUc6IO5ny5JKNkJAUEoiokQ1N60GDcHk0yboKQ&s=
>> > > 4y-EkuMJJlcYjI15KrZY8VE3eGEhkvg9ScDcHtItY2g&e=
>> > >
>> ====================================================
>> > ======
>> > > ====================
>> > > ---
>> clang-tools-extra/trunk/clang-tidy/modernize/
>> > > UseNullptrCheck.cpp (original)
>> > > +++
>> clang-tools-extra/trunk/clang-tidy/modernize/
>> > > UseNullptrCheck.cpp Wed Aug 19 12:50:22 2015
>> > > @@ -175,10 +175,10 @@ private:
>> > > class CastSequenceVisitor : public
>> > RecursiveASTVisitor
>> > > <CastSequenceVisitor> {
>> > > public:
>> > > CastSequenceVisitor(ASTContext &Context,
>> > > - SmallVector<StringRef,
>> 1>
>> > > UserNullMacros,
>> > > + ArrayRef<StringRef>
>> > UserNullMacros,
>> > > ClangTidyCheck &check)
>> > > : SM(Context.getSourceManager()),
>> Context
>> > (Context),
>> > > -
>> UserNullMacros(std::move(UserNullMacros)),
>> > Check
>> > > (check),
>> > > + UserNullMacros(UserNullMacros), Check
>> > (check),
>> > > FirstSubExpr(nullptr),
>> PruneSubtree(false)
>> > {}
>> > >
>> > > bool TraverseStmt(Stmt *S) {
>> > > @@ -435,7 +435,7 @@ private:
>> > > private:
>> > > SourceManager &SM;
>> > > ASTContext &Context;
>> > > - const SmallVector<StringRef, 1>
>> &UserNullMacros;
>> > > + ArrayRef<StringRef> UserNullMacros;
>> > > ClangTidyCheck &Check;
>> > > Expr *FirstSubExpr;
>> > > bool PruneSubtree;
>> > >
>> > >
>> _______________________________________________
>> > > cfe-commits mailing list
>> > > cfe-commits at lists.llvm.org
>> > > https://urldefense.proofpoint.com/v2/url?u=
>> > >
>> > http-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_cfe-2Dcommits
>> > > &d=BQIGaQ&c=eEvniauFctOgLOKGJOplqw&r=
>> > > 03tkj3107244TlY4t3_hEgkDY-UG6gKwwK0wOUS3qjM&m=
>> > > vEGzlOUc6IO5ny5JKNkJAUEoiokQ1N60GDcHk0yboKQ&s=
>> > > UUHB6kbqbYNAr9eIu4oMQTKtxnQ37-xusL7D-UY0GR4&e=
>> > >
>> > > _______________________________________________
>> > > cfe-commits mailing list
>> > > cfe-commits at lists.llvm.org
>> > > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150820/d6084fb6/attachment-0001.html>
More information about the cfe-commits
mailing list