anderslanglands added inline comments.
================ Comment at: clang/tools/c-index-test/c-index-test.c:1787 PrintCursor(cursor, NULL); + VisitorData *Data = (VisitorData *)d; PrintSingleTypeSize(T, " [type=%s] [typekind=%s]", " [sizeof=%lld]", ---------------- anderslanglands wrote: > aaron.ballman wrote: > > I'm not super familiar with c-index-test -- how certain are you that the > > client data will 1) be non-null, and 2) actually be a `VisitorData` pointer? > This was just copy-pasted from another function. I'm as certain as I can be > given that all the tests pass :\ From a quick scan, VisitorData is the only CXClientData struct used in the visitor callbacks, so I think we're safe. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138377/new/ https://reviews.llvm.org/D138377 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
