[PATCH] D13622: Add call to find_package to load LLVM dependencies

2015-10-10 Thread don hinton via cfe-commits
hintonda created this revision. hintonda added a reviewer: chapuni. hintonda added a subscriber: cfe-commits. ClangConfig requires LLVMConfig, so add find_package call in ClangConfig so find_package(clang REQUIRED CONFIG) will just work. This makes it easier for cmake based projects to use

Re: [PATCH] D13440: [clang-tidy] Python script for easy check rename

2015-10-10 Thread Piotr Zegar via cfe-commits
ClockMan added a comment. No, I don't have commits rights Repository: rL LLVM http://reviews.llvm.org/D13440 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12501: [clang-format] Obj-C dictionary literals: Fixed typecast getting put on a separate line from the key

2015-10-10 Thread Kent Sutherland via cfe-commits
ksuther added a comment. This patch is still awaiting a commit. Sorry to repeatedly post about this, just don't want it to get lost. Thanks! http://reviews.llvm.org/D12501 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D13344: Keep the IfStmt node even if the condition is invalid

2015-10-10 Thread Richard Smith via cfe-commits
rsmith added inline comments. Comment at: lib/Sema/SemaStmt.cpp:505-506 @@ -512,1 +504,4 @@ + } else { +ConditionExpr = new (Context) OpaqueValueExpr(SourceLocation(), + Context.VoidTy, VK_RValue); } Please

Re: [PATCH] D13344: Keep the IfStmt node even if the condition is invalid

2015-10-10 Thread Olivier Goffart via cfe-commits
ogoffart added a comment. Are the analysis run if there is an error? I think the consumer should expect null condition anyway. But i'll try to add an ErrorExpr anyway. http://reviews.llvm.org/D13344 ___ cfe-commits mailing list

[PATCH] D13620: Documentation for "Throw by value, catch by reference" check in clang-tidy

2015-10-10 Thread Tobias Langner via cfe-commits
randomcppprogrammer created this revision. randomcppprogrammer added reviewers: aaron.ballman, cfe-commits. adds documentation for the check mentioned in the title http://reviews.llvm.org/D13620 Files: docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst Index:

Re: [libcxx] r249936 - Get some of wchar_h.pass.cpp working on apple.

2015-10-10 Thread Richard Smith via cfe-commits
Thanks! On Oct 9, 2015 7:56 PM, "Eric Fiselier via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Author: ericwf > Date: Fri Oct 9 21:54:41 2015 > New Revision: 249936 > > URL: http://llvm.org/viewvc/llvm-project?rev=249936=rev > Log: > Get some of wchar_h.pass.cpp working on apple. > >