[PATCH] D41102: Setup clang-doc frontend framework

2018-03-05 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added inline comments. Comment at: clang-doc/BitcodeWriter.h:160 +class ClangDocBitcodeWriter { + public: + ClangDocBitcodeWriter(llvm::BitstreamWriter &Stream, Looks like Clang-format was applied incorrectly, because this is Google, not LLVM sty

[PATCH] D44120: [CFG] [analyzer] Heavier CFGValueTypedCall elements.

2018-03-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 137116. NoQ added a comment. Agreed to rename into `CFGCXXRecordTypedCall`. Because, yeah, `int` is a value as well. https://reviews.llvm.org/D44120 Files: include/clang/Analysis/CFG.h lib/Analysis/CFG.cpp lib/StaticAnalyzer/Core/ExprEngine.cpp lib/Sta

[PATCH] D44124: [analyzer] Support destruction and lifetime-extension of inlined function return values.

2018-03-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 137117. NoQ added a comment. Fix test comments and make them start from `0` :) Also rebase. https://reviews.llvm.org/D44124 Files: lib/StaticAnalyzer/Core/ExprEngineCXX.cpp test/Analysis/lifetime-extension.cpp Index: test/Analysis/lifetime-extension.cpp ==

[PATCH] D44129: [analyzer] NFC: Refactor the code for obtaining temporary lifetime info into a method.

2018-03-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 137119. NoQ added a comment. Rebase. https://reviews.llvm.org/D44129 Files: include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h lib/StaticAnalyzer/Core/ExprEngine.cpp lib/StaticAnalyzer/Core/ExprEngineCXX.cpp Index: lib/StaticAnalyzer/Core/ExprE

[PATCH] D44131: [analyzer] Support temporaries conjured by conservatively evaluated functions.

2018-03-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 137120. NoQ added a comment. Rebase. https://reviews.llvm.org/D44131 Files: lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp test/Analysis/explain-svals.cpp test/Analysis/temporaries.cpp Index: test/Analysis/temporaries.cpp ===

[PATCH] D42983: [clang-tidy] Add readability-simd-intrinsics check.

2018-03-05 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. In https://reviews.llvm.org/D42983#1025179, @alexfh wrote: > A late comment here: should this check start a new "portability" module? This > seems to be the main focus of the check rather than making code more readable. SG. Should I rename it? Repository: rCTE Clan

r326766 - [ExprConstant] Look through ExprWithCleanups for `allocsize`

2018-03-05 Thread George Burgess IV via cfe-commits
Author: gbiv Date: Mon Mar 5 23:42:36 2018 New Revision: 326766 URL: http://llvm.org/viewvc/llvm-project?rev=326766&view=rev Log: [ExprConstant] Look through ExprWithCleanups for `allocsize` Modified: cfe/trunk/lib/AST/ExprConstant.cpp cfe/trunk/test/CodeGenCXX/alloc-size.cpp Modified:

r326767 - Fix an unused variable warning; NFC

2018-03-05 Thread George Burgess IV via cfe-commits
Author: gbiv Date: Mon Mar 5 23:45:11 2018 New Revision: 326767 URL: http://llvm.org/viewvc/llvm-project?rev=326767&view=rev Log: Fix an unused variable warning; NFC Modified: cfe/trunk/lib/Analysis/CFG.cpp Modified: cfe/trunk/lib/Analysis/CFG.cpp URL: http://llvm.org/viewvc/llvm-project/c

<    1   2