falbrechtskirchinger added a comment.

In D127184#3572731 <https://reviews.llvm.org/D127184#3572731>, @nridge wrote:

> This change looks fine to me.
>
> I wonder though if we should be a bit more systematic about it, and try to 
> cover other newly added libstdc++ implementation headers?

Sure thing.

> (There are other new ones in gcc 11, and some in older versions that we've 
> missed, but I think handling a single gcc version would be a good scope for 
> this patch.)

I don't mind doing a systematic review including older and newer versions. Give 
me a few days to find some spare time to do it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127184

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D12718... Florian Albrechtskirchinger via Phabricator via cfe-commits
    • [PATCH] D... Florian Albrechtskirchinger via Phabricator via cfe-commits
    • [PATCH] D... Florian Albrechtskirchinger via Phabricator via cfe-commits
    • [PATCH] D... Florian Albrechtskirchinger via Phabricator via cfe-commits
    • [PATCH] D... Nathan Ridge via Phabricator via cfe-commits
    • [PATCH] D... Sam McCall via Phabricator via cfe-commits
    • [PATCH] D... Florian Albrechtskirchinger via Phabricator via cfe-commits

Reply via email to