[PATCH] D145365: [Tooling/Inclusion] Add missing index_sequence symbols.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 6 04:50:40 PST 2023


kadircet accepted this revision.
kadircet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/lib/Tooling/Inclusions/Stdlib/StdSpecialSymbolMap.inc:252
+// Remove them when the cppreference offline archive catches up.
+SYMBOL(index_sequence, std::, <utility>)
+SYMBOL(index_sequence_for, std::, <utility>)
----------------
hokein wrote:
> kadircet wrote:
> > there's also `integer_sequence` see https://eel.is/c++draft/utility.syn
> The integer_sequence has been indexed in the symbol page, it is already in `StdSymbolMap.inc`, https://github.com/llvm/llvm-project/blob/main/clang/lib/Tooling/Inclusions/Stdlib/StdSymbolMap.inc#L1576
oops, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145365



More information about the cfe-commits mailing list