[PATCH] D13000: [libclang] Expose AutoType

2015-09-20 Thread Sergey Kalinichev via cfe-commits
skalinichev created this revision. skalinichev added a subscriber: cfe-commits. This exposes the AutoType through LibClang interface http://reviews.llvm.org/D13000 Files: bindings/python/clang/cindex.py include/clang-c/Index.h test/Index/print-type.cpp tools/libclang/CXType.cpp Index: t

[PATCH] D13001: [libclang] Handle AutoType in clang_getTypeDeclaration

2015-09-20 Thread Sergey Kalinichev via cfe-commits
skalinichev created this revision. skalinichev added a subscriber: cfe-commits. Now that auto type is fixed by D11976, it also makes sense to support it in clang_getTypeDeclaration. I couldn't find any existing tests for this method, so no tests added... http://reviews.llvm.org/D13001 Files:

Re: [PATCH] D12895: Let cxa_demangle.cpp compile with gcc/libstdc++ 4.8 and clang-cl/MSVC2013's STL.

2015-09-20 Thread Marshall Clow via cfe-commits
mclow.lists accepted this revision. mclow.lists added a comment. This revision is now accepted and ready to land. This looks good to me. http://reviews.llvm.org/D12895 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

r248127 - Module debugging: Support submodules in the PCM/PCH debug info.

2015-09-20 Thread Adrian Prantl via cfe-commits
Author: adrian Date: Sun Sep 20 11:51:35 2015 New Revision: 248127 URL: http://llvm.org/viewvc/llvm-project?rev=248127&view=rev Log: Module debugging: Support submodules in the PCM/PCH debug info. Added: cfe/trunk/test/Modules/DebugInfoSubmodules.c cfe/trunk/test/Modules/Inputs/DebugSubmo

Re: [PATCH] D12839: Extend MoveConstructorInitCheck to also flag constructor arguments passed by value and can be moved assigned to fields.

2015-09-20 Thread Felix Berger via cfe-commits
flx updated this revision to Diff 35204. flx marked 10 inline comments as done. http://reviews.llvm.org/D12839 Files: clang-tidy/misc/MoveConstructorInitCheck.cpp clang-tidy/misc/MoveConstructorInitCheck.h clang-tidy/utils/CMakeLists.txt clang-tidy/utils/Matchers.h clang-tidy/utils/Type

Re: [PATCH] D12839: Extend MoveConstructorInitCheck to also flag constructor arguments passed by value and can be moved assigned to fields.

2015-09-20 Thread Felix Berger via cfe-commits
flx added inline comments. Comment at: clang-tidy/misc/MoveConstructorInitCheck.cpp:38 @@ +37,3 @@ + Node.isTriviallyCopyableType(Finder->getASTContext()) || + classHasTrivialCopyAndDestroy(Node)) { +return false; aaron.ballman wrote: > Why do you ne

[libcxxabi] r248129 - Let cxa_demangle.cpp compile with gcc/libstdc++ 4.8 and clang-cl/MSVC2013's STL.

2015-09-20 Thread Nico Weber via cfe-commits
Author: nico Date: Sun Sep 20 13:10:46 2015 New Revision: 248129 URL: http://llvm.org/viewvc/llvm-project?rev=248129&view=rev Log: Let cxa_demangle.cpp compile with gcc/libstdc++ 4.8 and clang-cl/MSVC2013's STL. libstdc++ needs a few typedefs in malloc_alloc. MSVC's STL needs rebind(), construct(

Re: [PATCH] D12895: Let cxa_demangle.cpp compile with gcc/libstdc++ 4.8 and clang-cl/MSVC2013's STL.

2015-09-20 Thread Nico Weber via cfe-commits
thakis closed this revision. thakis added a comment. r248129, thanks! http://reviews.llvm.org/D12895 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12761: MPI-Checker patch for Clang Static Analyzer

2015-09-20 Thread Alexander Droste via cfe-commits
Alexander_Droste marked 2 inline comments as done. Comment at: tools/clang/lib/StaticAnalyzer/Checkers/MPI-Checker/MPIBugReporter.cpp:137 @@ +136,3 @@ + + // This is never reached... + llvm::outs() << "" I figured out what the problem about the

r248137 - Don't pass StringRefs around by const reference. Pass by value instead per coding standards. NFC

2015-09-20 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sun Sep 20 19:20:04 2015 New Revision: 248137 URL: http://llvm.org/viewvc/llvm-project?rev=248137&view=rev Log: Don't pass StringRefs around by const reference. Pass by value instead per coding standards. NFC Modified: cfe/trunk/lib/Driver/Tools.cpp Modified: cfe/trunk

r248138 - Pass SourceLocation by value instead of by const reference since its just a 32-bit integer. NFC

2015-09-20 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sun Sep 20 20:23:32 2015 New Revision: 248138 URL: http://llvm.org/viewvc/llvm-project?rev=248138&view=rev Log: Pass SourceLocation by value instead of by const reference since its just a 32-bit integer. NFC Modified: cfe/trunk/lib/Sema/SemaOpenMP.cpp Modified: cfe/tru

[clang-tools-extra] r248139 - Pass SourceLocation by value instead of by const reference since its just a 32-bit integer. NFC

2015-09-20 Thread Craig Topper via cfe-commits
Author: ctopper Date: Sun Sep 20 20:33:03 2015 New Revision: 248139 URL: http://llvm.org/viewvc/llvm-project?rev=248139&view=rev Log: Pass SourceLocation by value instead of by const reference since its just a 32-bit integer. NFC Modified: clang-tools-extra/trunk/clang-modernize/Core/Transfo

Re: [PATCH] D10305: [Clang Static Analyzer] Bug identification

2015-09-20 Thread Babati Bence via cfe-commits
babati added a comment. Hi Sorry for the late answer. > Can you please rebase this patch based on http://reviews.llvm.org/D12673? Yes, I can. The patch will comes soon. > How do you expect this to work? i.e. would bug_id_1 always be generated along > with new improved bug_ids in the same pli

Re: [PATCH] D11182: [OPENMP 4.0] Initial support for 'omp declare reduction' construct.

2015-09-20 Thread Alexey Bataev via cfe-commits
ABataev added a comment. Ping http://reviews.llvm.org/D11182 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D10599: [OPENMP 4.0] Initial support for '#pragma omp declare simd' directive.

2015-09-20 Thread Alexey Bataev via cfe-commits
ABataev added a comment. Ping http://reviews.llvm.org/D10599 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r248142 - clang/test/Modules/DebugInfoSubmodules.c REQUIRES asserts due to -debug-only.

2015-09-20 Thread NAKAMURA Takumi via cfe-commits
Author: chapuni Date: Mon Sep 21 01:57:36 2015 New Revision: 248142 URL: http://llvm.org/viewvc/llvm-project?rev=248142&view=rev Log: clang/test/Modules/DebugInfoSubmodules.c REQUIRES asserts due to -debug-only. Modified: cfe/trunk/test/Modules/DebugInfoSubmodules.c Modified: cfe/trunk/test/