[PATCH] D54996: [libclang] Fix clang_Cursor_isAnonymous

2019-04-27 Thread Jorn Vernee via Phabricator via cfe-commits
JornVernee added a comment. Herald added a project: LLVM. This seems to have changed the behaviour w.r.t inline struct or union decls in fields and global variables e.g. struct foo { struct { int x; } bar; }; For the StructDecl cursor of 'bar' isAnonymous now returns t

[PATCH] D54996: [libclang] Fix clang_Cursor_isAnonymous

2019-01-10 Thread Ivan Donchevskii via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL350805: [libclang] Fix clang_Cursor_isAnonymous (authored by yvvan, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D54996?vs=181005&id=181008#

[PATCH] D54996: [libclang] Fix clang_Cursor_isAnonymous

2019-01-10 Thread Ivan Donchevskii via Phabricator via cfe-commits
yvvan updated this revision to Diff 181005. yvvan added a comment. Replace the absolute path with {{.*}} to be independent from the machine. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54996/new/ https://reviews.llvm.org/D54996 Files: test/Index/print-type.cpp tools/c-index-test/

[PATCH] D54996: [libclang] Fix clang_Cursor_isAnonymous

2019-01-09 Thread Ivan Donchevskii via Phabricator via cfe-commits
yvvan added a comment. Good point :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54996/new/ https://reviews.llvm.org/D54996 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D54996: [libclang] Fix clang_Cursor_isAnonymous

2019-01-09 Thread Nikolai Kosjar via Phabricator via cfe-commits
nik requested changes to this revision. nik added inline comments. This revision now requires changes to proceed. Comment at: test/Index/print-type.cpp:202 // CHECK: CallExpr=Bar:17:3 [type=outer::inner::Bar] [typekind=Elaborated] [canonicaltype=outer::inner::Bar] [canonicaltyp

[PATCH] D54996: [libclang] Fix clang_Cursor_isAnonymous

2018-11-28 Thread Ivan Donchevskii via Phabricator via cfe-commits
yvvan created this revision. yvvan added reviewers: nik, marcobubke. Herald added a subscriber: arphaman. Use the same logic as in TypePrinter::printTag to determine that the tag is anonymous and the separate check for namespaces. https://reviews.llvm.org/D54996 Files: test/Index/print-type.