[libcxx] r312892 - Fix PR34298 - Allow std::function with an incomplete return type.

2017-09-10 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Sun Sep 10 16:41:20 2017 New Revision: 312892 URL: http://llvm.org/viewvc/llvm-project?rev=312892=rev Log: Fix PR34298 - Allow std::function with an incomplete return type. This patch fixes llvm.org/PR34298. Previously libc++ incorrectly evaluated the __invokable trait via

[libcxx] r312891 - Revert "Fix PR34298 - Allow std::function with an incomplete return type."

2017-09-10 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Sun Sep 10 16:37:47 2017 New Revision: 312891 URL: http://llvm.org/viewvc/llvm-project?rev=312891=rev Log: Revert "Fix PR34298 - Allow std::function with an incomplete return type." This reverts commit r312890 because the test case fails to compile for older versions of

[PATCH] D37104: [libc++] PR34298: Change std::function constructor and move assignment operator SFINAE checks to allow std::function with an incomplete return type

2017-09-10 Thread Eric Fiselier via Phabricator via cfe-commits
EricWF commandeered this revision. EricWF edited reviewers, added: arphaman; removed: EricWF. EricWF added a comment. I committed a different version of this fix in r312890. Commandeering and closing this review. @arphaman Thanks for the initial patch. Repository: rL LLVM

[libcxx] r312890 - Fix PR34298 - Allow std::function with an incomplete return type.

2017-09-10 Thread Eric Fiselier via cfe-commits
Author: ericwf Date: Sun Sep 10 16:12:33 2017 New Revision: 312890 URL: http://llvm.org/viewvc/llvm-project?rev=312890=rev Log: Fix PR34298 - Allow std::function with an incomplete return type. This patch fixes llvm.org/PR34298. Previously libc++ incorrectly evaluated the __invokable trait via

[PATCH] D37643: Add objcImplementationDecl matcher

2017-09-10 Thread Dave Lee via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL312889: Add objcImplementationDecl matcher (authored by kastiglione). Changed prior to commit: https://reviews.llvm.org/D37643?vs=114442=114526#toc Repository: rL LLVM

r312889 - Add objcImplementationDecl matcher

2017-09-10 Thread Dave Lee via cfe-commits
Author: kastiglione Date: Sun Sep 10 14:00:15 2017 New Revision: 312889 URL: http://llvm.org/viewvc/llvm-project?rev=312889=rev Log: Add objcImplementationDecl matcher Summary: Add the `objcImplementationDecl` matcher. See related: D30854 Tested with: ```

[PATCH] D37667: [C++14][Sema] Disallow decltype(auto) deduction for lambdas

2017-09-10 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. `decltype(auto) lambda = []{};` and `decltype(auto) list = {0,1};` seem like very different situations to me; in the latter case, the initializer is not even an expression. The rule that bans `decltype([]{})` is that a *lambda-expression* shall not appear in an

Re: [libcxxabi] r308470 - Drop 'svn' suffix from version number.

2017-09-10 Thread Shoaib Meenai via cfe-commits
Note that Reid restored the old behavior in r312043. Configuring with -DLLVM_APPEND_VC_REV=OFF should also restore the old behavior, I believe. On 9/9/17, 1:55 PM, "cfe-commits on behalf of Dimitry Andric via cfe-commits"

Re: [clang-tools-extra] r312769 - Update for PrintHelpMessage not calling exit.

2017-09-10 Thread Alexander Kornienko via cfe-commits
On Fri, Sep 8, 2017 at 2:33 AM, Rafael Espindola via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rafael > Date: Thu Sep 7 17:33:39 2017 > New Revision: 312769 > > URL: http://llvm.org/viewvc/llvm-project?rev=312769=rev > Log: > Update for PrintHelpMessage not calling exit. > >

[PATCH] D37562: [X86] Lower _mm[256|512]_[mask[z]]_avg_epu[8|16] intrinsics to native llvm IR

2017-09-10 Thread Yael Tsafrir via Phabricator via cfe-commits
ytsafrir updated this revision to Diff 114523. ytsafrir added a comment. Removed the AVG builtins from ./include/clang/Basic/BuiltinsX86.def https://reviews.llvm.org/D37562 Files: include/clang/Basic/BuiltinsX86.def lib/Headers/avx2intrin.h lib/Headers/avx512bwintrin.h

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-10 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor updated this revision to Diff 114524. teemperor added a comment. - Moved code from FrontendAction.cpp into createFileManager() https://reviews.llvm.org/D37416 Files: include/clang/Frontend/CompilerInstance.h lib/Frontend/CompilerInstance.cpp lib/Frontend/FrontendAction.cpp

[PATCH] D37413: [X86][MS-InlineAsm] Extended support for variables / identifiers on memory / immediate expressions

2017-09-10 Thread coby via Phabricator via cfe-commits
coby updated this revision to Diff 114520. coby added a comment. Herald added a subscriber: eraman. addressed Simon's comments Repository: rL LLVM https://reviews.llvm.org/D37413 Files: include/clang/Parse/Parser.h include/clang/Sema/Sema.h lib/Parse/ParseStmtAsm.cpp

r312882 - [clang][SemaStmtAsm] small refactoring, NFC.

2017-09-10 Thread Coby Tayree via cfe-commits
Author: coby Date: Sun Sep 10 05:39:21 2017 New Revision: 312882 URL: http://llvm.org/viewvc/llvm-project?rev=312882=rev Log: [clang][SemaStmtAsm] small refactoring, NFC. Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp URL:

[PATCH] D37668: [X86][intrinsics] lower _mm[256|512]_mask[z]_set1_epi[8|16|32|64] intrinsic to IR

2017-09-10 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added a comment. As with https://reviews.llvm.org/D37562, strip the builtins from include/clang/Basic/BuiltinsX86.def https://reviews.llvm.org/D37668 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D37668: [X86][intrinsics] lower _mm[256|512]_mask[z]_set1_epi[8|16|32|64] intrinsic to IR

2017-09-10 Thread jina via Phabricator via cfe-commits
jina.nahias created this revision. this is clang part , the llvm part is https://reviews.llvm.org/differential/diff/114515/ https://reviews.llvm.org/D37668 Files: lib/Headers/avx512bwintrin.h lib/Headers/avx512fintrin.h lib/Headers/avx512vlbwintrin.h lib/Headers/avx512vlintrin.h

[PATCH] D37667: [C++14][Sema] Disallow decltype(auto) deduction for lambdas

2017-09-10 Thread Blitz Rakete via Phabricator via cfe-commits
Rakete created this revision. `decltype(auto) lambda = []{};` is currently allowed by clang, gcc and MSVC, but it is actually not valid, because `decltype([]{})` (which is how the type of `lambda` is deduced) is ill-formed, I think. I honestly could argue both ways (and I did once), but

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-10 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor updated this revision to Diff 114504. teemperor added a comment. Herald added a subscriber: mgorny. - Added test case. https://reviews.llvm.org/D37416 Files: lib/Frontend/CompilerInstance.cpp unittests/Frontend/CMakeLists.txt unittests/Frontend/CompilerInstanceTest.cpp Index:

r312877 - set the svn:executable property, seems that it is necessary for apache (discussed with Tanya by email)

2017-09-10 Thread Sylvestre Ledru via cfe-commits
Author: sylvestre Date: Sun Sep 10 01:00:03 2017 New Revision: 312877 URL: http://llvm.org/viewvc/llvm-project?rev=312877=rev Log: set the svn:executable property, seems that it is necessary for apache (discussed with Tanya by email) Modified: cfe/trunk/www/menu.html.incl (props changed)