kadircet added a comment. In D73451#1841595 <https://reviews.llvm.org/D73451#1841595>, @hokein wrote:
> In D73451#1841510 <https://reviews.llvm.org/D73451#1841510>, @kadircet wrote: > > > we seem to be missing some symbols now, like `polymorphic_allocator` > > `random_shuffle` `gets` etc. > > > yeap, two majoir reasons: > > - some symbols <https://en.cppreference.com/w/cpp/symbol_index/zombie_names> > are deprecated/removed in new C++ standard, e.g. random_shuffle is removed in > C++17 > - missing information in cppreference page, e.g. there is no header section > for `polymorphic_allocator` in the offline cppreference page :( > > Regarding those deprecated symbols, we could handle them according to > which C++ standard we are using, as we have enough information from > cppreferences <https://en.cppreference.com/w/cpp/symbol_index/zombie_names>, > but it needs some work, and is low priority. Ah I see, it is sad that cppreference just deletes those from symbol index, instead of marking them as `delete after c++XX`, I am not sure when the symbols introduced by this patch are added(but it is likely that they are introduced with c++20), so I think losing existing symbols has a worse effect than not having information regarding c++20 symbols. Because these are symbols that are likely to be used by current c++ developers and supported by clangd, and it will be a regression to drop those, whereas the newly introduced symbols are already missing. So maybe postpone this update until we implement parsing of zombie_names page as well, also please file a bug either way so that we don't lose track. > > >> and there seems to be a functional change to the parser. Are they >> intentional, if so why? > > The change should be NFC, it just makes the parser more robust on invalid > links, the new version of the cppreference have invalid links on the symbol > index page. > updated the code to make it clearer. maybe separate that out into a different patch so that it stands out? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73451/new/ https://reviews.llvm.org/D73451 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits