r275127 - Fix test submitted with r275115 (failed on ppc64 buildbots).

2016-07-11 Thread Wolfgang Pieb via cfe-commits
Author: wolfgangp Date: Mon Jul 11 18:20:28 2016 New Revision: 275127 URL: http://llvm.org/viewvc/llvm-project?rev=275127&view=rev Log: Fix test submitted with r275115 (failed on ppc64 buildbots). Modified: cfe/trunk/test/CodeGen/forwarding-blocks-if.c Modified: cfe/trunk/test/CodeGen/forwar

r275128 - Correcting the previous fix for test submitted with r275115.

2016-07-11 Thread Wolfgang Pieb via cfe-commits
Author: wolfgangp Date: Mon Jul 11 18:27:19 2016 New Revision: 275128 URL: http://llvm.org/viewvc/llvm-project?rev=275128&view=rev Log: Correcting the previous fix for test submitted with r275115. Modified: cfe/trunk/test/CodeGen/forwarding-blocks-if.c Modified: cfe/trunk/test/CodeGen/forwar

Buildbot numbers for the week of 6/26/2016 - 7/02/2016

2016-07-11 Thread Galina Kistanova via cfe-commits
Hello everyone, Below are some buildbot numbers for the week of 6/26/2016 - 7/02/2016. Please see the same data in attached csv files: The longest time each builder was red during the last week; "Status change ratio" by active builder (percent of builds that changed the builder status from greed

Buildbot numbers for the week of 7/03/2016 - 7/09/2016

2016-07-11 Thread Galina Kistanova via cfe-commits
Hello everyone, Below are some buildbot numbers for the last week of 7/03/2016 - 7/09/2016. Please see the same data in attached csv files: The longest time each builder was red during the last week; "Status change ratio" by active builder (percent of builds that changed the builder status from

[PATCH] D22248: [Sema] Create a separate group for incompatible function pointer warning

2016-07-11 Thread Bruno Cardoso Lopes via cfe-commits
bruno created this revision. bruno added reviewers: bob.wilson, doug.gregor, krememek. bruno added subscribers: cfe-commits, dexonsmith. Give incompatible pointer warning for function pointers its own diagnostic group while leaving it as a subgroup of incompatible-pointer-types. This is in prepa

Re: r274991 - [clang-cl] Add support for /Zd

2016-07-11 Thread Saleem Abdulrasool via cfe-commits
On Mon, Jul 11, 2016 at 12:29 PM, David Majnemer via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > > On Mon, Jul 11, 2016 at 12:18 PM, Nico Weber wrote: > >> On Mon, Jul 11, 2016 at 12:19 PM, David Majnemer < >> david.majne...@gmail.com> wrote: >> >>> >>> >>> On Mon, Jul 11, 2016 at 9:03 A

Re: [PATCH] D22189: llvm.noalias - Clang CodeGen - check restrict variable map only for restrict-qualified lvalues

2016-07-11 Thread David Majnemer via cfe-commits
majnemer added a subscriber: majnemer. majnemer accepted this revision. majnemer added a reviewer: majnemer. majnemer added a comment. This revision is now accepted and ready to land. LGTM Comment at: lib/CodeGen/CodeGenFunction.h:2725 @@ -2724,3 +2724,3 @@ void EmitStoreOfSc

[PATCH] D22255: [libcxx] Create __is_alloc_constructible trait which checks the validity of allocator_traits construction.

2016-07-11 Thread Eric Fiselier via cfe-commits
EricWF created this revision. EricWF added a reviewer: mclow.lists. EricWF added a subscriber: cfe-commits. Example Usage: To check the `EmplaceConstructible` Container requirement you would ask: `__is_alloc_constructible::value` http://reviews.llvm.org/D22255 Files: include/memory includ

Re: [PATCH] D21706: [libcxx] refactor for throw

2016-07-11 Thread Weiming Zhao via cfe-commits
weimingz added a comment. Hi Marshall, do you have any comments? http://reviews.llvm.org/D21706 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D21823: [Driver] Add flags for enabling both types of PGO Instrumentation

2016-07-11 Thread Jake VanAdrighem via cfe-commits
jakev updated the summary for this revision. jakev updated this revision to Diff 63634. jakev added a comment. Change patch to use `-fprofile-generate` to enable IRPGO. Repository: rL LLVM http://reviews.llvm.org/D21823 Files: include/clang/Driver/Options.td lib/Driver/Tools.cpp test/D

Re: [PATCH] D21904: [OpenMP] Initial implementation of parse+sema for clause use_device_ptr of 'target data'

2016-07-11 Thread Alexey Bataev via cfe-commits
ABataev added a comment. I don't see successful test for the clause, which prints AST and checks serialization/deserialization. Also it should include a template test to check that dependent objects are handled correctly Comment at: include/clang/AST/OpenMPClause.h:4236 @@ +42

Re: [PATCH] D22255: [libcxx] Create __is_alloc_constructible trait which checks the validity of allocator_traits construction.

2016-07-11 Thread Eric Fiselier via cfe-commits
EricWF updated this revision to Diff 63635. EricWF added a comment. - Don't use `is_constructible` in the `__is_alloc_constructible` implementation. `is_constructible` requires `is_destructible`, but that isn't required within allocators. Instead this trait checks the validity of the placemen

Re: [PATCH] D22255: [libcxx] Create __is_alloc_constructible trait which checks the validity of allocator_traits construction.

2016-07-11 Thread Eric Fiselier via cfe-commits
EricWF updated this revision to Diff 63636. EricWF added a comment. Add correct tests. http://reviews.llvm.org/D22255 Files: include/memory include/type_traits include/vector test/libcxx/utilities/memory/allocator.traits/__is_alloc_constructible.pass.cpp test/std/containers/sequences

Re: [PATCH] D22073: libc++: test lock-free atomic alignment

2016-07-11 Thread Eric Fiselier via cfe-commits
EricWF added a comment. - The test should be moved to `test/libcxx/atomics/atomics.align` since it's libc++ specific. - Please give the anonymous struct declarations unique names. `T1`, `T2`, ..., `TN` is fine. Currently they all mangle to `main::type` in diagnostic output. - The test fails to l

Re: [PATCH] D22242: [analyzer] De-duplicate some code for discovering the origin of the symbol.

2016-07-11 Thread Anna Zaks via cfe-commits
zaks.anna added a subscriber: zaks.anna. Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/SymExpr.h:81 @@ -78,2 +80,3 @@ unsigned computeComplexity() const; + const MemRegion *getOriginRegion() const; }; Please, add a doxygen style comment. http:

r275154 - [MS ABI] Support throwing/catching __unaligned types

2016-07-11 Thread David Majnemer via cfe-commits
Author: majnemer Date: Mon Jul 11 23:42:50 2016 New Revision: 275154 URL: http://llvm.org/viewvc/llvm-project?rev=275154&view=rev Log: [MS ABI] Support throwing/catching __unaligned types We need to mark the appropriate bits in ThrowInfo and HandlerType so that the personality routine can correct

Re: [PATCH] D22222: [ASTMatchers] isSignedInteger() and isUnsignedInteger()

2016-07-11 Thread Clement Courbet via cfe-commits
courbet updated this revision to Diff 63644. courbet added a comment. Regenerate doc. http://reviews.llvm.org/D2 Files: docs/LibASTMatchersReference.html include/clang/ASTMatchers/ASTMatchers.h lib/ASTMatchers/Dynamic/Registry.cpp unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp

Re: [PATCH] D22222: [ASTMatchers] isSignedInteger() and isUnsignedInteger()

2016-07-11 Thread Clement Courbet via cfe-commits
courbet added a comment. Thanks. http://reviews.llvm.org/D2 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r275157 - [ASTMatchers] isSignedInteger() and isUnsignedInteger()

2016-07-11 Thread Clement Courbet via cfe-commits
Author: courbet Date: Tue Jul 12 01:36:00 2016 New Revision: 275157 URL: http://llvm.org/viewvc/llvm-project?rev=275157&view=rev Log: [ASTMatchers] isSignedInteger() and isUnsignedInteger() Complementary to isInteger(), these match signed and unsigned integers respectively. Review: http://review

<    1   2