[PATCH] D60687: [clangd] Check file path of declaring header when deciding whether to insert include.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 15 04:42:50 PDT 2019


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

Previously, we would use include spelling of the declaring header to check
whether the inserted header is the same as the main file. This doesn't help because
we only have file path of the main file.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D60687

Files:
  clangd/CodeComplete.cpp
  clangd/Headers.cpp
  clangd/Headers.h
  clangd/IncludeFixer.cpp
  unittests/clangd/HeadersTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60687.195136.patch
Type: text/x-patch
Size: 7758 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190415/d635d6cb/attachment-0001.bin>


More information about the cfe-commits mailing list