sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
Thanks! IIUC, this already fixes the cases we'd seen of include-insertion of a header into itself. Is it feasible to add a test case for that? ================ Comment at: clangd/Headers.h:140 /// - /// \param DeclaringHeader is the original header corresponding to \p + /// \param DeclaringHeader is path of the original header corresponding to \p /// InsertedHeader e.g. the header that declares a symbol. ---------------- is the path Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60687/new/ https://reviews.llvm.org/D60687 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
