[PATCH] D24082: [CMake] Fix libc++abi arm build w/o libunwind.

2016-11-12 Thread Eric Fiselier via cfe-commits
EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. Nope LGTM. https://reviews.llvm.org/D24082 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo

r286716 - Add support of the next Ubuntu (17.04 - Zesty Zapus)

2016-11-12 Thread Sylvestre Ledru via cfe-commits
Author: sylvestre Date: Sat Nov 12 03:26:30 2016 New Revision: 286716 URL: http://llvm.org/viewvc/llvm-project?rev=286716&view=rev Log: Add support of the next Ubuntu (17.04 - Zesty Zapus) Modified: cfe/trunk/lib/Driver/ToolChains.cpp Modified: cfe/trunk/lib/Driver/ToolChains.cpp URL: http

r286717 - Add the new option -Og in the release notes. See r286602 for more information

2016-11-12 Thread Sylvestre Ledru via cfe-commits
Author: sylvestre Date: Sat Nov 12 04:37:17 2016 New Revision: 286717 URL: http://llvm.org/viewvc/llvm-project?rev=286717&view=rev Log: Add the new option -Og in the release notes. See r286602 for more information Modified: cfe/trunk/docs/ReleaseNotes.rst Modified: cfe/trunk/docs/ReleaseNote

r286718 - As we released 3.9, from the 4.0 release notes, points to version 3.9 instead of 3.8

2016-11-12 Thread Sylvestre Ledru via cfe-commits
Author: sylvestre Date: Sat Nov 12 04:38:18 2016 New Revision: 286718 URL: http://llvm.org/viewvc/llvm-project?rev=286718&view=rev Log: As we released 3.9, from the 4.0 release notes, points to version 3.9 instead of 3.8 Modified: cfe/trunk/docs/ReleaseNotes.rst Modified: cfe/trunk/docs/Rel

[PATCH] D24639: [Sema] Warn when returning a lambda that captures a local variable by reference

2016-11-12 Thread Erik Pilkington via cfe-commits
erik.pilkington updated this revision to Diff 77717. erik.pilkington added a comment. This new patch rewrites the check to just use the captures in the lambda's CXXRecordDecl, instead of searching through the returned expression to find the LambdaExpr. Thanks! https://reviews.llvm.org/D24639

[PATCH] D24639: [Sema] Warn when returning a lambda that captures a local variable by reference

2016-11-12 Thread Erik Pilkington via cfe-commits
erik.pilkington added inline comments. Comment at: test/SemaCXX/cxx1y-generic-lambdas-capturing.cpp:185 f(y, d); - f(z, d); + f(z, d); // expected-warning {{address of stack memory associated with local variable 'z' returned}} decltype(a)

r286721 - Use descriptive message if list initializer is incorrectly parenthesized.

2016-11-12 Thread Serge Pavlov via cfe-commits
Author: sepavloff Date: Sat Nov 12 09:38:55 2016 New Revision: 286721 URL: http://llvm.org/viewvc/llvm-project?rev=286721&view=rev Log: Use descriptive message if list initializer is incorrectly parenthesized. If initializer contains parentheses around braced list where it is not allowed, as in c

[PATCH] D25816: Use descriptive message if list initializer is incorrectly parenthesized.

2016-11-12 Thread Serge Pavlov via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL286721: Use descriptive message if list initializer is incorrectly parenthesized. (authored by sepavloff). Changed prior to commit: https://reviews.llvm.org/D25816?vs=75404&id=77725#toc Repository: r

[PATCH] D26435: Use unique_ptr for cached tokens for default arguments in C++.

2016-11-12 Thread Malcolm Parsons via cfe-commits
malcolm.parsons requested changes to this revision. malcolm.parsons added a comment. This revision now requires changes to proceed. Expected Passes: 4977 Expected Failures : 18 Unsupported Tests : 25 Unexpected Failures: 5037 Needs more work? https://reviews.llvm.org/D26435 ___

[PATCH] D26582: [clang-tidy] Handle template instantiations in modenize-use-default check

2016-11-12 Thread Malcolm Parsons via cfe-commits
malcolm.parsons created this revision. malcolm.parsons added reviewers: aaron.ballman, alexfh. malcolm.parsons added a subscriber: cfe-commits. Duplicate fixes were being created for explicit template instantiations of out-of-line constructors or destructors. Fixes PR30921. https://reviews.llvm

[PATCH] D26196: Add support for non-zero null pointer for C and OpenCL

2016-11-12 Thread John McCall via cfe-commits
rjmccall added a comment. In https://reviews.llvm.org/D26196#592823, @yaxunl wrote: > Hi John, > > It seems the casting from a pointer to different address space is not > affected by this change. When a null pointer is casted to different address > space, it is casted the same way as an ordinar

r286733 - [AVX-512] Use scalar vfmsub/vfnmsub mask3 intrinsics instead of inverting the mask argument of a vfmadd intrinsic.

2016-11-12 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sat Nov 12 17:24:34 2016 New Revision: 286733 URL: http://llvm.org/viewvc/llvm-project?rev=286733&view=rev Log: [AVX-512] Use scalar vfmsub/vfnmsub mask3 intrinsics instead of inverting the mask argument of a vfmadd intrinsic. Summary: Inverting the mask argument does not r

[PATCH] D26019: [AVX-512] Use scalar vfmsub/vfnmsub mask3 intrinsics instead of inverting the mask argument of a vfmadd intrinsic.

2016-11-12 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL286733: [AVX-512] Use scalar vfmsub/vfnmsub mask3 intrinsics instead of inverting the… (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D26019?vs=75978&id=77729#toc Repository:

r286735 - Link static PIE programs against rcrt0.o on OpenBSD

2016-11-12 Thread Brad Smith via cfe-commits
Author: brad Date: Sat Nov 12 17:52:03 2016 New Revision: 286735 URL: http://llvm.org/viewvc/llvm-project?rev=286735&view=rev Log: Link static PIE programs against rcrt0.o on OpenBSD Patch by Stefan Kempf. Modified: cfe/trunk/lib/Driver/Tools.cpp cfe/trunk/test/Driver/openbsd.c Modified

RE: r286714 - [AVX-512] Convert the rest of the masked shift by immediate and by single element builtins over to the newly added unmasked builtins and a select.

2016-11-12 Thread Yung, Douglas via cfe-commits
Hi Craig, One of our internal tests noticed a potential issue with your change that I wanted to see if you were aware of in the file avx512bwintrin.h. Specifically, you converted _mm512_slli_epi16, _mm512_srai_epi16, _mm512_srli_epi16 from macros into actual functions, but the functions don't

r286736 - Revert r286735 due to test failure

2016-11-12 Thread Brad Smith via cfe-commits
Author: brad Date: Sat Nov 12 18:16:21 2016 New Revision: 286736 URL: http://llvm.org/viewvc/llvm-project?rev=286736&view=rev Log: Revert r286735 due to test failure Modified: cfe/trunk/lib/Driver/Tools.cpp cfe/trunk/test/Driver/openbsd.c Modified: cfe/trunk/lib/Driver/Tools.cpp URL: ht

r286738 - [AVX-512] Add returns to shift intrinsics that converted from macros in r286714.

2016-11-12 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sat Nov 12 18:35:01 2016 New Revision: 286738 URL: http://llvm.org/viewvc/llvm-project?rev=286738&view=rev Log: [AVX-512] Add returns to shift intrinsics that converted from macros in r286714. Modified: cfe/trunk/lib/Headers/avx512bwintrin.h Modified: cfe/trunk/lib/Head

r286741 - [AMDGPU] Add f16 builtin functions (VI+)

2016-11-12 Thread Konstantin Zhuravlyov via cfe-commits
Author: kzhuravl Date: Sat Nov 12 20:37:05 2016 New Revision: 286741 URL: http://llvm.org/viewvc/llvm-project?rev=286741&view=rev Log: [AMDGPU] Add f16 builtin functions (VI+) Differential Revision: https://reviews.llvm.org/D26476 Added: cfe/trunk/test/SemaOpenCL/builtins-amdgcn-error-f16.cl

[PATCH] D26476: [AMDGPU] Add support for f16 builtin functions for VI+

2016-11-12 Thread Konstantin Zhuravlyov via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL286741: [AMDGPU] Add f16 builtin functions (VI+) (authored by kzhuravl). Changed prior to commit: https://reviews.llvm.org/D26476?vs=77654&id=77732#toc Repository: rL LLVM https://reviews.llvm.org/D

[PATCH] D26196: Add support for non-zero null pointer for C and OpenCL

2016-11-12 Thread Yaxun Liu via cfe-commits
yaxunl added a comment. >> It seems the casting from a pointer to different address space is not >> affected by this change. When a null pointer is casted to different address >> space, it is casted the same way as an ordinary pointer, e.g. by >> addrspacecast. > > You mean, the code-generatio

[PATCH] D26196: Add support for non-zero null pointer for C and OpenCL

2016-11-12 Thread Yaxun Liu via cfe-commits
yaxunl added a comment. >> It seems the casting from a pointer to different address space is not >> affected by this change. When a null pointer is casted to different address >> space, it is casted the same way as an ordinary pointer, e.g. by >> addrspacecast. > > You mean, the code-generatio

r286748 - Fix PR28366: Handle variables from enclosing local scopes more gracefully during constant expression evaluation.

2016-11-12 Thread Faisal Vali via cfe-commits
Author: faisalv Date: Sun Nov 13 00:09:16 2016 New Revision: 286748 URL: http://llvm.org/viewvc/llvm-project?rev=286748&view=rev Log: Fix PR28366: Handle variables from enclosing local scopes more gracefully during constant expression evaluation. Only look for a variable's value in the constant

[PATCH] D23493: Fix PR28366: Teach the const-expression evaluator to be more fault tolerant with non-const enclosing local variables, or otherwise fold them if const.

2016-11-12 Thread Faisal Vali via cfe-commits
faisalv closed this revision. faisalv added a comment. Fixed by commit: r286748 http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20161107/176865.html Thanks for the review! https://reviews.llvm.org/D23493 ___ cfe-commits mailing list cfe-comm

r286752 - Bitcode: Change module reader functions to return an llvm::Expected.

2016-11-12 Thread Peter Collingbourne via cfe-commits
Author: pcc Date: Sun Nov 13 01:00:17 2016 New Revision: 286752 URL: http://llvm.org/viewvc/llvm-project?rev=286752&view=rev Log: Bitcode: Change module reader functions to return an llvm::Expected. Differential Revision: https://reviews.llvm.org/D26562 Modified: cfe/trunk/lib/CodeGen/CGOpen

[PATCH] D26546: [PPC] Add vec_insert4b/vec_extract4b to altivec.h

2016-11-12 Thread Kit Barton via cfe-commits
kbarton added inline comments. Comment at: lib/Headers/altivec.h:12014 +#define vec_insert4b(__a, __b, __c) \ + ((vector unsigned char)__builtin_vsx_xxinsertw((__a), (__b), (__c) & 0xF)) +#endif nemanjai wrote: > As far as I can tell by looking at this patch and

r286756 - [X86] Remove extra escaped new lines in intrinsic headers left over from an earlier conversion away from a macro. NFC

2016-11-12 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sun Nov 13 01:26:31 2016 New Revision: 286756 URL: http://llvm.org/viewvc/llvm-project?rev=286756&view=rev Log: [X86] Remove extra escaped new lines in intrinsic headers left over from an earlier conversion away from a macro. NFC Modified: cfe/trunk/lib/Headers/avx512bw

r286757 - [AVX-512] Replace masked dword and qword variable shift builtins with unmasked builtins and a select.

2016-11-12 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sun Nov 13 01:26:34 2016 New Revision: 286757 URL: http://llvm.org/viewvc/llvm-project?rev=286757&view=rev Log: [AVX-512] Replace masked dword and qword variable shift builtins with unmasked builtins and a select. This is part of a set of changes to allow InstCombine in the