Re: [PATCH] D6551: Improvements to scan-build.

2015-09-16 Thread Sylvestre Ledru via cfe-commits
sylvestre.ledru closed this revision. sylvestre.ledru added a comment. To close this review. http://reviews.llvm.org/D6551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D6551: Improvements to scan-build.

2015-09-02 Thread Антон Ярцев via cfe-commits
ayartsev added a comment. Removed trailing whitespaces and committed as r246710. Thanks for reviews and assistance! http://reviews.llvm.org/D6551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

Re: [PATCH] D6551: Improvements to scan-build.

2015-08-28 Thread Anna Zaks via cfe-commits
zaks.anna added a comment. Please, make sure to include a very specific commit title/message. http://reviews.llvm.org/D6551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D6551: Improvements to scan-build.

2015-08-28 Thread Devin Coughlin via cfe-commits
dcoughlin accepted this revision. dcoughlin added a comment. I ran our build-bot scripts with this and it looks good to me, although I've noted two places with trailing whitespace. Thanks for your patience! Comment at: tools/scan-build/scan-build:1403 @@ +1402,3 @@ + my $Args

Re: [PATCH] D6551: Improvements to scan-build.

2015-08-28 Thread Sylvestre Ledru via cfe-commits
sylvestre.ledru added a comment. Anton, I think you should just commit it. We can always iterate on this later if it breaks anything. The patch is big and rebasing is time consuming. http://reviews.llvm.org/D6551 ___ cfe-commits mailing list cfe-co

Re: [PATCH] D6551: Improvements to scan-build.

2015-08-24 Thread Антон Ярцев via cfe-commits
ayartsev updated this revision to Diff 32969. ayartsev added a comment. Updated the patch with r245621. OK to commit? http://reviews.llvm.org/D6551 Files: tools/scan-build/scan-build Index: tools/scan-build/scan-build === --- to

Re: [PATCH] D6551: Improvements to scan-build.

2015-08-11 Thread Антон Ярцев via cfe-commits
ayartsev added a reviewer: zaks.anna. ayartsev removed a subscriber: zaks.anna. ayartsev updated this revision to Diff 31868. ayartsev added a comment. Hi Honggyu, thanks for updating the patch! The new patch contains changes introduced by r244400 (http://reviews.llvm.org/D10356) and r244673 pl

RE: [PATCH] D6551: Improvements to scan-build.

2015-08-09 Thread Honggyu Kim via cfe-commits
mmits@lists.llvm.org; kreme...@apple.com; > zaks.a...@gmail.com > Subject: Re: [PATCH] D6551: Improvements to scan-build. > > honggyu.kim added a comment. > > Hi all, > > I have also tested this patch by analyzing ARM version of Linux kernel with > my patch > http:/

Re: [PATCH] D6551: Improvements to scan-build.

2015-08-07 Thread Sylvestre Ledru via cfe-commits
sylvestre.ledru added a comment. (if you commit it, please update the description, "Improvements to scan-build" is too general http://reviews.llvm.org/D6551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailm