r260874 - clangIndex requires LLVMIR as Core, since r260858 uses llvm/IR.

2016-02-14 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Sun Feb 14 22:29:36 2016 New Revision: 260874 URL: http://llvm.org/viewvc/llvm-project?rev=260874&view=rev Log: clangIndex requires LLVMIR as Core, since r260858 uses llvm/IR. Modified: cfe/trunk/lib/Index/CMakeLists.txt Modified: cfe/trunk/lib/Index/CMakeLists.txt URL:

[clang-tools-extra] r260873 - [clang-tidy] ClangTidy check to flag uninitialized builtin and pointer fields.

2016-02-14 Thread Felix Berger via cfe-commits
Author: flx Date: Sun Feb 14 22:27:56 2016 New Revision: 260873 URL: http://llvm.org/viewvc/llvm-project?rev=260873&view=rev Log: [clang-tidy] ClangTidy check to flag uninitialized builtin and pointer fields. Summary: This patch is a continuation of http://reviews.llvm.org/D10553 by Jonathan B C

Re: [PATCH] D16517: ClangTidy check to flag uninitialized builtin and pointer fields.

2016-02-14 Thread Felix Berger via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL260873: [clang-tidy] ClangTidy check to flag uninitialized builtin and pointer fields. (authored by flx). Changed prior to commit: http://reviews.llvm.org/D16517?vs=47757&id=47942#toc Repository: rL

r260872 - Add isAnyPointer() matchers. Register missing matchers.

2016-02-14 Thread Felix Berger via cfe-commits
Author: flx Date: Sun Feb 14 22:00:39 2016 New Revision: 260872 URL: http://llvm.org/viewvc/llvm-project?rev=260872&view=rev Log: Add isAnyPointer() matchers. Register missing matchers. Summary: The isAnyPointer() matcher is useful for http://reviews.llvm.org/D15623. Reviewers: alexfh, klimek S

Re: [PATCH] D17064: Only invoke ForRangeCopyCheck on expensive-to-copy types.

2016-02-14 Thread Felix Berger via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL260870: [clang-tidy] Only invoke ForRangeCopyCheck on expensive-to-copy types. (authored by flx). Changed prior to commit: http://reviews.llvm.org/D17064?vs=47419&id=47940#toc Repository: rL LLVM ht

[clang-tools-extra] r260870 - [clang-tidy] Only invoke ForRangeCopyCheck on expensive-to-copy types.

2016-02-14 Thread Felix Berger via cfe-commits
Author: flx Date: Sun Feb 14 21:36:23 2016 New Revision: 260870 URL: http://llvm.org/viewvc/llvm-project?rev=260870&view=rev Log: [clang-tidy] Only invoke ForRangeCopyCheck on expensive-to-copy types. Summary: Fix oversight not checking the value of the Optional returned by isExpensiveToCopy().

[clang-tools-extra] r260869 - Improve documentation

2016-02-14 Thread Felix Berger via cfe-commits
Author: flx Date: Sun Feb 14 21:27:54 2016 New Revision: 260869 URL: http://llvm.org/viewvc/llvm-project?rev=260869&view=rev Log: Improve documentation Modified: clang-tools-extra/trunk/docs/clang-tidy/checks/performance-for-range-copy.rst Modified: clang-tools-extra/trunk/docs/clang-tidy/

r260867 - Sema: prevent assertion on stack return checking

2016-02-14 Thread Saleem Abdulrasool via cfe-commits
Author: compnerd Date: Sun Feb 14 19:51:24 2016 New Revision: 260867 URL: http://llvm.org/viewvc/llvm-project?rev=260867&view=rev Log: Sema: prevent assertion on stack return checking In the case that the array indexing itself is within a type dependent context, bail out of the evaluation. We wo

r260866 - [AST/index] Introduce an option 'SuppressTemplateArgsInCXXConstructors' in printing policy.

2016-02-14 Thread Argyrios Kyrtzidis via cfe-commits
Author: akirtzidis Date: Sun Feb 14 19:32:36 2016 New Revision: 260866 URL: http://llvm.org/viewvc/llvm-project?rev=260866&view=rev Log: [AST/index] Introduce an option 'SuppressTemplateArgsInCXXConstructors' in printing policy. Enable it for USRs and names when indexing. Forward references can

r260864 - Sema: constify EvalAddr, EvalVal

2016-02-14 Thread Saleem Abdulrasool via cfe-commits
Author: compnerd Date: Sun Feb 14 18:36:49 2016 New Revision: 260864 URL: http://llvm.org/viewvc/llvm-project?rev=260864&view=rev Log: Sema: constify EvalAddr, EvalVal Propagate const throughout these methods as they are non-mutating analyzers of state. NFC. Modified: cfe/trunk/lib/Sema/Sem

r260865 - silence -Wreturn-type warnings

2016-02-14 Thread Saleem Abdulrasool via cfe-commits
Author: compnerd Date: Sun Feb 14 18:36:52 2016 New Revision: 260865 URL: http://llvm.org/viewvc/llvm-project?rev=260865&view=rev Log: silence -Wreturn-type warnings These codepaths would generate warnings with GCC on linux even though the switch was covered. Add llvm_unreachable markers to indi

Re: [PATCH] D16360: unordered_map: Avoid unnecessary mallocs when no insert occurs

2016-02-14 Thread Duncan P. N. Exon Smith via cfe-commits
dexonsmith updated this revision to Diff 47936. dexonsmith added a comment. Updated the patch to apply against r260513. The logic is much simpler now; thanks Eric for the cleanup. Let me know if there's anything else to do! http://reviews.llvm.org/D16360 Files: include/__hash_table test/

Re: r260847 - c-index-test: Fix libdeps corresponding to r260841.

2016-02-14 Thread NAKAMURA Takumi via cfe-commits
BUILD_SHARED_LIBS requires them. Especially, Win32 DLL (PRIVATE) is most strict. CMake's transitive (PUBLIC/INTERFACE) libdeps sometimes misses incomplete deps. To make sure libdeps to be fine, I think we need to run a Win32 DLL builder. It requires not msvc but cygming. http://bb.pgr.jp/builders/

[PATCH] D17248: [Sema] PR26444 fix crash when alignment value is <= 2**16

2016-02-14 Thread don hinton via cfe-commits
hintonda created this revision. hintonda added a reviewer: majnemer. hintonda added a subscriber: cfe-commits. Sema allows max values up to 2**28, use unsigned instead of unsiged short to hold values that large. http://reviews.llvm.org/D17248 Files: lib/CodeGen/CGValue.h test/Sema/attr-align

r260861 - [test/Index] Set a specific target for the test.

2016-02-14 Thread Argyrios Kyrtzidis via cfe-commits
Author: akirtzidis Date: Sun Feb 14 16:38:38 2016 New Revision: 260861 URL: http://llvm.org/viewvc/llvm-project?rev=260861&view=rev Log: [test/Index] Set a specific target for the test. Modified: cfe/trunk/test/Index/Core/index-source.m Modified: cfe/trunk/test/Index/Core/index-source.m URL:

r260858 - [index] Factor libclang's functionality to determing the mangled name of symbols into the clangIndex library.

2016-02-14 Thread Argyrios Kyrtzidis via cfe-commits
Author: akirtzidis Date: Sun Feb 14 16:30:14 2016 New Revision: 260858 URL: http://llvm.org/viewvc/llvm-project?rev=260858&view=rev Log: [index] Factor libclang's functionality to determing the mangled name of symbols into the clangIndex library. Added: cfe/trunk/include/clang/Index/CodegenN

Re: [PATCH] D16264: For FreeBSD, use _p variants of libraries for linking C++ programs

2016-02-14 Thread Dimitry Andric via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL260851: As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD, (authored by dim). Changed prior to commit: http://reviews.llvm.org/D16264?vs=47913&id=47930#toc Repository: rL LLVM h

Re: r260201 - [CMake] Providing a CMake cache for 3-stage builds

2016-02-14 Thread Duncan P. N. Exon Smith via cfe-commits
> On 2016-Feb-08, at 22:01, Chris Bieneman via cfe-commits > wrote: > > Author: cbieneman > Date: Tue Feb 9 00:01:47 2016 > New Revision: 260201 > > URL: http://llvm.org/viewvc/llvm-project?rev=260201&view=rev > Log: > [CMake] Providing a CMake cache for 3-stage builds > > This cache file ca

Re: r260496 - [Objective-c] Stop attaching section "datacoal_nt" to global variables.

2016-02-14 Thread Duncan P. N. Exon Smith via cfe-commits
> On 2016-Feb-10, at 22:36, Akira Hatanaka via cfe-commits > wrote: > > Author: ahatanak > Date: Thu Feb 11 00:36:35 2016 > New Revision: 260496 > > URL: http://llvm.org/viewvc/llvm-project?rev=260496&view=rev > Log: > [Objective-c] Stop attaching section "datacoal_nt" to global variables. >

Re: [libcxx] r260514 - Teach __hash_table how to handle unordered_map's __hash_value_type.

2016-02-14 Thread Duncan P. N. Exon Smith via cfe-commits
(For anyone else trying to line up this commit message with the patch: it looks like this message is a dup of r260513, and this patch is one of its described follow-ups: > On 2016-Feb-11, at 04:25, Eric Fiselier via cfe-commits > wrote: > > The following changes are planed for future revisions:

r260856 - Fix some typos in the clang doc

2016-02-14 Thread Sylvestre Ledru via cfe-commits
Author: sylvestre Date: Sun Feb 14 14:20:58 2016 New Revision: 260856 URL: http://llvm.org/viewvc/llvm-project?rev=260856&view=rev Log: Fix some typos in the clang doc Modified: cfe/trunk/docs/ClangFormatStyleOptions.rst cfe/trunk/docs/CrossCompilation.rst cfe/trunk/docs/LanguageExten

Re: r260847 - c-index-test: Fix libdeps corresponding to r260841.

2016-02-14 Thread Argyrios Kyrtzidis via cfe-commits
Which configuration required them ? I was under the impression CMake is passing the transitive libdeps. > On Feb 14, 2016, at 1:19 AM, NAKAMURA Takumi via cfe-commits > wrote: > > Author: chapuni > Date: Sun Feb 14 03:19:04 2016 > New Revision: 260847 > > URL: http://llvm.org/viewvc/llvm-proje

Re: r260850 - Don't leak the ASTUnit when done with testing.

2016-02-14 Thread Argyrios Kyrtzidis via cfe-commits
Thanks! > On Feb 14, 2016, at 5:18 AM, Benjamin Kramer via cfe-commits > wrote: > > Author: d0k > Date: Sun Feb 14 07:18:06 2016 > New Revision: 260850 > > URL: http://llvm.org/viewvc/llvm-project?rev=260850&view=rev > Log: > Don't leak the ASTUnit when done with testing. > > Found by lsan. >

Re: r260851 - As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD,

2016-02-14 Thread Dimitry Andric via cfe-commits
On 14 Feb 2016, at 17:08, Dimitry Andric via cfe-commits wrote: > > Author: dim > Date: Sun Feb 14 10:08:20 2016 > New Revision: 260851 > > URL: http://llvm.org/viewvc/llvm-project?rev=260851&view=rev > Log: > As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD, > C++ program

r260851 - As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD,

2016-02-14 Thread Dimitry Andric via cfe-commits
Author: dim Date: Sun Feb 14 10:08:20 2016 New Revision: 260851 URL: http://llvm.org/viewvc/llvm-project?rev=260851&view=rev Log: As reported in https://llvm.org/bugs/show_bug.cgi?id=25496, on FreeBSD, C++ programs compiled for profiling (using `-pg`) should be linked with `-lc++_p` (or `-lstdc++_

Re: [PATCH] Add ObjCBoxable handling to ObjCMigrator

2016-02-14 Thread Alex Denisov via cfe-commits
Rebased against trunk: objc_migrator_objc_boxable.patch Description: Binary data -- AlexDenisov Software Engineer, http://lowlevelbits.org > On 05 Dec 2015, at 22:19, John McCall wrote: > >> On Dec 5, 2015, at 7:58 AM, AlexDenisov <1101.deb...@gmail.com> wrote: >> Extend ObjCMigrator to cov

r260850 - Don't leak the ASTUnit when done with testing.

2016-02-14 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sun Feb 14 07:18:06 2016 New Revision: 260850 URL: http://llvm.org/viewvc/llvm-project?rev=260850&view=rev Log: Don't leak the ASTUnit when done with testing. Found by lsan. Modified: cfe/trunk/tools/c-index-test/core_main.cpp Modified: cfe/trunk/tools/c-index-test/core_ma

Re: r260842 - [index] Enhance c-index-test tool and have it link and test the clangIndex library directly.

2016-02-14 Thread NAKAMURA Takumi via cfe-commits
Okay, I won't object as far as it were safe that libclang (many clang/llvm libraries are linked) and other static libraries could stand side-by-side. I supposed c-index-test is the tool to run and check libclang. FYI, you should specify libraries that are used in the target. Updated in r260847.

r260847 - c-index-test: Fix libdeps corresponding to r260841.

2016-02-14 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Sun Feb 14 03:19:04 2016 New Revision: 260847 URL: http://llvm.org/viewvc/llvm-project?rev=260847&view=rev Log: c-index-test: Fix libdeps corresponding to r260841. Modified: cfe/trunk/tools/c-index-test/CMakeLists.txt Modified: cfe/trunk/tools/c-index-test/CMakeLists.tx