[PATCH] D42575: [clangd] Better handling symbols defined in macros.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 26 05:50:37 PST 2018


hokein created this revision.
hokein added a reviewer: ioeric.
Herald added subscribers: jkorous-apple, ilya-biryukov, klimek.

For symbols defined inside macros:

- use expansion location, if the symbol is formed via macro concatenation.
- use spelling location, otherwise.

This will fix some symbols that have ill-format location (especial invalid filepath).


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42575

Files:
  clangd/index/SymbolCollector.cpp
  unittests/clangd/Annotations.cpp
  unittests/clangd/Annotations.h
  unittests/clangd/SymbolCollectorTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42575.131578.patch
Type: text/x-patch
Size: 6955 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180126/840f7fda/attachment.bin>


More information about the cfe-commits mailing list