ioeric added a comment. Looks great! Thanks for doing this.
Could you also check in the tool that is used to generate the mapping? We need a way to update the mapping when cppreference is updated. ================ Comment at: clangd/index/CanonicalIncludes.cpp:123 static const std::vector<std::pair<const char *, const char *>> SystemHeaderMap = { ---------------- Can we remove the suffix header mapping now? Is it for the `std::chrono::` symbols? What are the reasons not to include them in this patch? Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58345/new/ https://reviews.llvm.org/D58345 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits