ldionne added a comment.
Herald added a subscriber: usaxena95.
Herald added a project: clang-tools-extra.
This is awesome, but the script doesn't work anymore with `-language=cpp` -- is
there any interest in looking into that? The C++ Standard Library has changed
quite a bit recently and it woul
This revision was automatically updated to reflect the committed changes.
Closed by commit rL364044: [clangd] Add include-mapping for C symbols.
(authored by hokein, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.l
kadircet accepted this revision.
kadircet added inline comments.
This revision is now accepted and ready to land.
Comment at: clang-tools-extra/clangd/include-mapping/gen_std.py:95
- parse_pages = [
-(cpp_root, "symbol_index.html", "std::"),
-# std sub-namespace symbo
hokein added inline comments.
Comment at: clang-tools-extra/clangd/include-mapping/gen_std.py:90
+symbol_index_root = page_root
+parse_pages = [(page_root, "index.html", "")]
+
kadircet wrote:
> maybe we should rather pass some something like "INVALID" as
hokein updated this revision to Diff 204731.
hokein marked 7 inline comments as done.
hokein added a comment.
Address review comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63270/new/
https://reviews.llvm.org/D63270
Files:
clang-tools-ex
kadircet added inline comments.
Comment at: clang-tools-extra/clangd/include-mapping/cppreference_parser.py:1
-#!/usr/bin/env python
-#===- gen_std.py - --*- python
-*--===#
could we add a similar License and header comme
hokein created this revision.
hokein added a reviewer: kadircet.
Herald added subscribers: jfb, arphaman, jkorous, MaskRay, ilya-biryukov.
Herald added a project: clang.
This resolves the issue of introducing c++-style includes for C files.
- refactor the gen_std.py, make it reusable for parsing