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&view=rev
> Log:
> Get some of wchar_h.pass.cpp working on apple.
>
>
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
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 clan
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
cfe-commits@lists.
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
http://lists.llvm.o
ogoffart updated this revision to Diff 37035.
ogoffart added a comment.
The updated patch uses OpaqueValueExpr
http://reviews.llvm.org/D13344
Files:
lib/Sema/SemaStmt.cpp
test/Misc/ast-dump-invalid.cpp
Index: test/Misc/ast-dump-invalid.cpp
==
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