[PATCH] D21290: [include-fixer] only deduplicate symbols after matching symbols with the undefined identifier.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 13 07:51:33 PDT 2016


ioeric created this revision.
ioeric added a reviewer: bkramer.
ioeric added a subscriber: cfe-commits.

we should only deduplicate symbols after matching symbols with the
undefined identifier. This patch tries to fix the situation where matching
symbols are deleted when it has the same header as a non-matching symbol, which
can prevent finding the correct header even if there is a matched symbol.

http://reviews.llvm.org/D21290

Files:
  include-fixer/SymbolIndexManager.cpp
  unittests/include-fixer/IncludeFixerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21290.60525.patch
Type: text/x-patch
Size: 4722 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160613/15359201/attachment-0001.bin>


More information about the cfe-commits mailing list