RedX2501 added a comment.
Has this been merged yet?
https://reviews.llvm.org/D10833
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RedX2501 added a comment.
Ping?
https://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RedX2501 added a subscriber: klimek.
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RedX2501 added a reviewer: klimek.
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
RedX2501 added inline comments.
Comment at: include/clang-c/Index.h:3815-3823
@@ -3814,2 +3814,11 @@
/**
+ * \brief Returns true if a variable with function scope is a non-static local
variable.
+ */
+CINDEX_LINKAGE bool clang_Cursor_hasLocalStorage(CXCursor C);
+
+/*
+ * \bri
RedX2501 updated this revision to Diff 39158.
RedX2501 added a comment.
Added module tests to libclang and python.
http://reviews.llvm.org/D10833
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_cursor.py
include/clang-c/Index.h
include/clang/AST/OperationKinds.h
RedX2501 retitled this revision from "Added functions to retrieve information
about variable storage in libclang and its python bindings." to "Added
functions to retrieve information about whether a vardecl is local in libclang
and its python bindings.".
RedX2501 updated this revision to Diff 39
RedX2501 marked 2 inline comments as done.
Comment at: bindings/python/tests/cindex/test_cursor.py:333
@@ +332,3 @@
+# not exposed yet
+# ".*" : BinaryOperator.PtrMemD,
+"->*" : BinaryOperator.PtrMemI,
kfunk wrote:
> What about this? How is
RedX2501 updated this revision to Diff 39176.
RedX2501 added a comment.
Changed points raised during review.
http://reviews.llvm.org/D10833
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_cursor.py
include/clang-c/Index.h
include/clang/AST/OperationKinds.h
tes
RedX2501 added a reviewer: doug.gregor.
RedX2501 added a comment.
The code owners file says you are the owner of all parts that do not belong to
somebody else. So i hope it is ok if i attach you as reviewer to this.
http://reviews.llvm.org/D10833
_
RedX2501 added a reviewer: doug.gregor.
RedX2501 added a comment.
The code owners file says you are the owner of all parts that do not belong to
somebody else. So i hope it is ok if i attach you as reviewer to this.
http://reviews.llvm.org/D10834
_
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
RedX2501 accepted this revision.
RedX2501 added a reviewer: RedX2501.
RedX2501 added a comment.
This revision is now accepted and ready to land.
This was accepted into r219809.
http://reviews.llvm.org/D5538
___
cfe-commits mailing list
cfe-commits@l
RedX2501 closed this revision.
RedX2501 added a comment.
Accepted into r219809.
http://reviews.llvm.org/D5538
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RedX2501 updated this revision to Diff 52358.
RedX2501 added a comment.
Fixed compiler warnings.
http://reviews.llvm.org/D10834
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_cursor.py
include/clang-c/Index.h
test/Index/islocalvardecl.cpp
tools/c-index-test/c
RedX2501 updated this revision to Diff 52367.
RedX2501 added a comment.
Fix compilation error introduced due to breaking a long string.
Fixed indentation.
http://reviews.llvm.org/D10834
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_cursor.py
include/clang-c/Inde
RedX2501 created this revision.
RedX2501 added reviewers: EricWF, MatzeB, echristo, jroelofs.
RedX2501 added a subscriber: cfe-commits.
http://reviews.llvm.org/D18752
Files:
docs/CommandGuide/lit.rst
Index: docs/CommandGuide/lit.rst
=
RedX2501 updated this revision to Diff 52530.
RedX2501 added a comment.
Added reference to testing infrastructure document.
http://reviews.llvm.org/D18752
Files:
docs/CommandGuide/lit.rst
Index: docs/CommandGuide/lit.rst
===
---
RedX2501 updated this revision to Diff 52531.
RedX2501 added a comment.
I had a hard time myself figuring out what tmpbase and tmpdir are. I have used
your wording.
Somebody else needs to commit this as I don't have any rights for it.
http://reviews.llvm.org/D18752
Files:
docs/CommandGuide/
RedX2501 added a comment.
Thanks for the advice.
Please use Guilherme Bufolo as the name for the commit.
Am 04.04.2016 21:33 schrieb "Paul Robinson" <
paul_robin...@playstation.sony.com>:
> probinson added a subscriber: probinson.
> probinson added a comment.
>
> In http://reviews.llvm.org/
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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/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
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
RedX2501 added a comment.
Ping
http://reviews.llvm.org/D10834
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
RedX2501 updated this revision to Diff 42267.
RedX2501 added a comment.
Added explicit integer assignment to enum to emphasize api contract behaviour.
http://reviews.llvm.org/D10833
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_cursor.py
include/clang-c/Index.h
RedX2501 added a comment.
In http://reviews.llvm.org/D10833#306281, @akyrtzi wrote:
> The patch causes tests to fail, please look into it.
Where can i see the test reports? I'm having trouble running the test on my
side on windows (VS2013).
http://reviews.llvm.org/D10833
_
RedX2501 updated this revision to Diff 42386.
RedX2501 added a comment.
Added functions to libclang.exports.
http://reviews.llvm.org/D10833
Files:
bindings/python/clang/cindex.py
bindings/python/tests/cindex/test_cursor.py
include/clang-c/Index.h
include/clang/AST/OperationKinds.h
tes
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
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
44 matches
Mail list logo