r281197 - Attempt #3 to placate MSVC.

2016-09-11 Thread Richard Smith via cfe-commits
Author: rsmith Date: Mon Sep 12 01:38:31 2016 New Revision: 281197 URL: http://llvm.org/viewvc/llvm-project?rev=281197&view=rev Log: Attempt #3 to placate MSVC. Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp URL:

r281195 - Attempt #2 to placate MSVC

2016-09-11 Thread Richard Smith via cfe-commits
Author: rsmith Date: Mon Sep 12 01:23:26 2016 New Revision: 281195 URL: http://llvm.org/viewvc/llvm-project?rev=281195&view=rev Log: Attempt #2 to placate MSVC Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp URL:

r281194 - Attempt to placate MSVC.

2016-09-11 Thread Richard Smith via cfe-commits
Author: rsmith Date: Mon Sep 12 01:13:44 2016 New Revision: 281194 URL: http://llvm.org/viewvc/llvm-project?rev=281194&view=rev Log: Attempt to placate MSVC. Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp Modified: cfe/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp URL: ht

Re: [PATCH] D24446: Implement revision to LWG#2758

2016-09-11 Thread Marshall Clow via cfe-commits
mclow.lists added inline comments. Comment at: include/string:615 @@ +614,3 @@ + +// template +// struct __can_be_converted_to_string_view : public _LIBCPP_BOOL_CONSTANT( These bits will be removed. Comment at: test/std/strings/basic.string/str

r281189 - [MS ABI] Add /include directives for dynamic TLS

2016-09-11 Thread David Majnemer via cfe-commits
Author: majnemer Date: Sun Sep 11 21:51:43 2016 New Revision: 281189 URL: http://llvm.org/viewvc/llvm-project?rev=281189&view=rev Log: [MS ABI] Add /include directives for dynamic TLS MSVC emits /include directives in the .drective section for the __dyn_tls_init function (decorated as ___dyn_tls_

Re: [PATCH] D21968: [libcxx] Externally threaded libc++ variant - Take 2

2016-09-11 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL281179: [libcxx] Introduce an externally-threaded libc++ variant. (authored by asiri). Changed prior to commit: https://reviews.llvm.org/D21968?vs=70840&id=70969#toc Repository: rL LLVM https://revi

[libcxx] r281179 - [libcxx] Introduce an externally-threaded libc++ variant.

2016-09-11 Thread Asiri Rathnayake via cfe-commits
Author: asiri Date: Sun Sep 11 16:46:40 2016 New Revision: 281179 URL: http://llvm.org/viewvc/llvm-project?rev=281179&view=rev Log: [libcxx] Introduce an externally-threaded libc++ variant. This patch further decouples libc++ from pthread, allowing libc++ to be built against other threading syste

Re: [PATCH] D18462: Fix for clang_Cursor_getSpellingNameRange()

2016-09-11 Thread Milian Wolff via cfe-commits
milianw added a comment. ping, any update on this? https://reviews.llvm.org/D18462 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D24444: [clang-tidy] modernize-use-default default constructor bugfix

2016-09-11 Thread Malcolm Parsons via cfe-commits
malcolm.parsons created this revision. malcolm.parsons added a reviewer: klimek. malcolm.parsons added subscribers: cfe-commits, klimek, alexfh. Only member initializers that are written should prevent using '= default' on a default constructor. https://reviews.llvm.org/D2 Files: clang-tid

Re: [PATCH] D24426: DebugInfo: Pass non-zero alignment to DIBuilder only if aligment was forced

2016-09-11 Thread Victor Leschuk via cfe-commits
vleschuk retitled this revision from "DebugInfo: use llvm::DINode::FlagAlignment to mark forcibly aligned data" to "DebugInfo: Pass non-zero alignment to DIBuilder only if aligment was forced". vleschuk updated the summary for this revision. vleschuk updated this revision to Diff 70965. vleschuk

Re: [PATCH] D24395: Align declarations that are preceded by different number of commas.

2016-09-11 Thread Nikola Smiljanić via cfe-commits
nikola abandoned this revision. nikola added a comment. Thanks for letting me know, that patch looks more complete so I'll abandon this. I hope it lands soon! https://reviews.llvm.org/D24395 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

Re: [PATCH] D21279: Fix some issues in clang-format's AlignConsecutive modes

2016-09-11 Thread Nikola Smiljanić via cfe-commits
nikola added a subscriber: nikola. Comment at: lib/Format/WhitespaceManager.cpp:356 @@ -273,3 +355,3 @@ // If there is more than one matching token per line, or if the number of // preceding commas, or the scope depth, do not match anymore, end the // sequence.