[PATCH] D52313: [clang-doc] Avoid parsing undefined base classes

2018-10-03 Thread Julie Hockett via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL343703: [clang-doc] Avoid parsing undefined base classes (authored by juliehockett, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D52313?vs=1

[PATCH] D52313: [clang-doc] Avoid parsing undefined base classes

2018-09-21 Thread Julie Hockett via Phabricator via cfe-commits
juliehockett updated this revision to Diff 166526. juliehockett added a comment. Adding test case https://reviews.llvm.org/D52313 Files: clang-tools-extra/clang-doc/Serialize.cpp clang-tools-extra/test/clang-doc/bc-record.cpp clang-tools-extra/test/clang-doc/mapper-record.cpp clang-tool

[PATCH] D52313: [clang-doc] Avoid parsing undefined base classes

2018-09-20 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri accepted this revision. lebedev.ri added a comment. This revision is now accepted and ready to land. If it **no longer** crashes, i guess you can test for that? Other than that, SG. https://reviews.llvm.org/D52313 ___ cfe-commits mailing

[PATCH] D52313: [clang-doc] Avoid parsing undefined base classes

2018-09-20 Thread Julie Hockett via Phabricator via cfe-commits
juliehockett created this revision. juliehockett added reviewers: jakehehrlich, leonardchan, lebedev.ri. juliehockett added a project: clang-tools-extra. Don't try to parse base classes for declarations that are not definitions (segfaults, as there is no DefinitionData to access). https://revie