hokein added a comment.

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.

> 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.


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

Reply via email to