[PATCH] D61316: [clangd] Improvements to header mapping: more precise parsing of cppreference symbol pages.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 30 08:16:54 PDT 2019


sammccall created this revision.
sammccall added a reviewer: kadircet.
Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov.
Herald added a project: clang.

Previously we were just jumping from the symbol index to the symbol page, and
grabbing all the headers mentioned there. But the page often lists multiple
symbols, and so we got false positives and thus ambiguities (which were dropped).

Now we look at which declarations are for the symbol we want, and prefer headers
listed above that symbol. If there are none, we fall back to the old behavior.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D61316

Files:
  clangd/StdSymbolMap.inc
  clangd/include-mapping/gen_std.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61316.197337.patch
Type: text/x-patch
Size: 11318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190430/91febd1f/attachment-0001.bin>


More information about the cfe-commits mailing list