r256135 - [Sema] Use range-based for loops. NFC

2015-12-20 Thread Craig Topper via cfe-commits
Author: ctopper Date: Mon Dec 21 00:35:56 2015 New Revision: 256135 URL: http://llvm.org/viewvc/llvm-project?rev=256135&view=rev Log: [Sema] Use range-based for loops. NFC Modified: cfe/trunk/lib/Sema/SemaExpr.cpp Modified: cfe/trunk/lib/Sema/SemaExpr.cpp URL: http://llvm.org/viewvc/llvm-pr

Re: [PATCH] D10833: Retrieve BinaryOperator::getOpcode and BinaryOperator::getOpcodeStr via libclang and its python interface

2015-12-20 Thread guibufolo+l...@gmail.com via cfe-commits
RedX2501 added a comment. Ping http://reviews.llvm.org/D10833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r256134 - [Cygwin] Enable TLS as emutls.

2015-12-20 Thread NAKAMURA Takumi via cfe-commits
I will remove it a few days after, when I made sure it were not required any more. I have a commit for this in my local branch. 2015-12-21 11:44 GMT+09:00 Rafael Espíndola : > Can we delete > > config.available_features.add('tls') > > now? > > On 20 December 2015 at 21:37, NAKAMURA Takumi via cfe

Re: r256134 - [Cygwin] Enable TLS as emutls.

2015-12-20 Thread Rafael Espíndola via cfe-commits
Can we delete config.available_features.add('tls') now? On 20 December 2015 at 21:37, NAKAMURA Takumi via cfe-commits wrote: > Author: chapuni > Date: Sun Dec 20 20:37:23 2015 > New Revision: 256134 > > URL: http://llvm.org/viewvc/llvm-project?rev=256134&view=rev > Log: > [Cygwin] Enable TLS a

r256134 - [Cygwin] Enable TLS as emutls.

2015-12-20 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Sun Dec 20 20:37:23 2015 New Revision: 256134 URL: http://llvm.org/viewvc/llvm-project?rev=256134&view=rev Log: [Cygwin] Enable TLS as emutls. It resolves clang selfhosting with std::once() for Cygwin. FIXME: It may be EmulatedTLS-generic also for X86-Android. FIXME: Pass E

[clang-tools-extra] r256133 - check-clang-tools requires clang-headers due to test/clang-tidy.

2015-12-20 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Sun Dec 20 19:35:28 2015 New Revision: 256133 URL: http://llvm.org/viewvc/llvm-project?rev=256133&view=rev Log: check-clang-tools requires clang-headers due to test/clang-tidy. clang-modernize provided clang-headers but it was removed. Modified: clang-tools-extra/trunk/

Re: [PATCH] Install cmake files to lib/cmake/clang

2015-12-20 Thread Niels Ole Salscheider via cfe-commits
On Monday 21 December 2015, 00:10:10 CET, you wrote: > This is the right location for platform-specific files. > > Also, search for LLVM's CMake files in this directory. The corresponding LLVM patch can be found at http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20151214/320963.html >

[PATCH] Install cmake files to lib/cmake/clang

2015-12-20 Thread Niels Ole Salscheider via cfe-commits
This is the right location for platform-specific files. Also, search for LLVM's CMake files in this directory. --- CMakeLists.txt | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 2c54e75..e4a839d 100644 --- a/CMakeLists.txt +++ b

Re: [PATCH] D15613: Fix ARM __cxa_end_cleanup() and gc-sections.

2015-12-20 Thread Logan Chien via cfe-commits
logan added a comment. @echristo: IMO, I prefer not to add such comment for two reasons: 1. It will obscure the main idea of this piece of code. 2. It is common to specify the section flags (compilers will emit these flags by default as well), so it is unlikely to be removed in the future. Bes

Re: [PATCH] D9600: Add scan-build python implementation

2015-12-20 Thread Ryan Govostes via cfe-commits
rgov added a subscriber: rgov. rgov added a comment. I tried out intercept-build and found it very easy to use. I tried running it on projects that compile with both make and xcodebuild and it worked for both. Thanks for contributing this tool. http://reviews.llvm.org/D9600

Re: [clang-tools-extra] r255765 - [clang-tidy] Don't use diag() for debug output

2015-12-20 Thread Tom Hulton-Harrop via cfe-commits
I just got the latest llvm svn revision and when I run clang-tidy with -checks="readability-braces-around-statements" I get this error: Assertion failed: (CondEndLoc.isValid()), function findRParenLoc, file ../llvm/tools/clang/tools/extra/clang-tidy/readability/ BracesAroundStatementsCheck.cpp,

r256122 - [X86] Add missing m64/int64 conversions

2015-12-20 Thread Michael Kuperstein via cfe-commits
Author: mkuper Date: Sun Dec 20 06:37:18 2015 New Revision: 256122 URL: http://llvm.org/viewvc/llvm-project?rev=256122&view=rev Log: [X86] Add missing m64/int64 conversions Define the 64-bit equivalents of _m_to_int and _m_from_int. Differential Revision: http://reviews.llvm.org/D15572 Modifie

r256121 - [X86] Add signed aliases for popcnt intrinsics

2015-12-20 Thread Michael Kuperstein via cfe-commits
Author: mkuper Date: Sun Dec 20 06:35:35 2015 New Revision: 256121 URL: http://llvm.org/viewvc/llvm-project?rev=256121&view=rev Log: [X86] Add signed aliases for popcnt intrinsics The Intel manual documents both an unsigned form (_mm_popcnt_u32) and a signed form (_popcnt32) of the intrinsic. Add