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

Reply via email to