[PATCH] D27049: [OpenCL] Refactor out ReadPipe/WritePipe

2016-11-26 Thread Yaron Keren via Phabricator via cfe-commits
yaron.keren added inline comments. Comment at: lib/AST/ASTContext.cpp:8258 return RHS; -return isa(LHS) ? getReadPipeType(ResultType) - : getWritePipeType(ResultType); +const PipeType *PT = LHS->getAs(); +return

r287990 - [DOXYGEN] Updated instruction names corresponding to avxintrin.h intrinsics.

2016-11-26 Thread Ekaterina Romanova via cfe-commits
Author: kromanova Date: Sat Nov 26 13:38:19 2016 New Revision: 287990 URL: http://llvm.org/viewvc/llvm-project?rev=287990=rev Log: [DOXYGEN] Updated instruction names corresponding to avxintrin.h intrinsics. Documentation for some of the avxintrin.h's intrinsics errorneously said that non

[libcxx] r287988 - Implement conjuntion/disjuntion/negation for LFTS v2. Same code and tests for the ones in std::

2016-11-26 Thread Marshall Clow via cfe-commits
Author: marshall Date: Sat Nov 26 12:45:03 2016 New Revision: 287988 URL: http://llvm.org/viewvc/llvm-project?rev=287988=rev Log: Implement conjuntion/disjuntion/negation for LFTS v2. Same code and tests for the ones in std:: Added:

[PATCH] D21298: [Clang-tidy] delete null check

2016-11-26 Thread Gergely Angeli via Phabricator via cfe-commits
SilverGeri updated this revision to Diff 79338. SilverGeri added a comment. Herald added a subscriber: JDevlieghere. only warn, not fix when the 'if' statement has 'else' clause keeping comments https://reviews.llvm.org/D21298 Files: clang-tidy/readability/CMakeLists.txt

[libcxx] r287981 - Implement the 'detection idiom' from LFTS v2

2016-11-26 Thread Marshall Clow via cfe-commits
Author: marshall Date: Sat Nov 26 09:49:40 2016 New Revision: 287981 URL: http://llvm.org/viewvc/llvm-project?rev=287981=rev Log: Implement the 'detection idiom' from LFTS v2 Added: libcxx/trunk/test/std/experimental/utilities/meta/meta.detect/

[PATCH] D27116: Fix crash when using __has_nothrow_copy with inherited constructors

2016-11-26 Thread Philippe Daouadi via Phabricator via cfe-commits
blastrock abandoned this revision. blastrock added a comment. No, https://reviews.llvm.org/D23765 seems to fix this issue. https://reviews.llvm.org/D27116 ___ cfe-commits mailing list cfe-commits@lists.llvm.org