[PATCH] D119130: [clangd] NFC: Move stdlib headers handling to Clang

Wed Feb 9 02:04:33 PST 2022

kbobyrev added a comment.

In D119130#3307222 <https://reviews.llvm.org/D119130#3307222>, @sammccall wrote:

> In D119130#3304505 <https://reviews.llvm.org/D119130#3304505>, @kbobyrev wrote:
>
>> Oh, and also regarding the Python generator scripts: they don't seem to work with the latest archives :( 2018 mentioned in the docs (works perfectly) but 2019 crashes.
>
> Yikes, we should fix that. Especially since the 2019 version is 3 years out of date already...

Agreed! Though, the other problem is that it seems that nothing beyond 2019 version really exists. Which means that even if we support that one, it's unlikely to be up-to-date. For us to be relatively up-to-date we might need to ask someone working on cppreference for new archives and the format is likely to be different from 2019, too, so I don't know if we should fix the issue now vs. try to get newer archives and fix the issue after that.

>> Probably not very relevant if we switch to libc++ eventually.
>
> This seems pretty unlikely to me.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119130/new/

https://reviews.llvm.org/D119130