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

2016-06-13 Thread Eric Liu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL272576: [include-fixer] only deduplicate symbols after matching symbols with theā€¦ (authored by ioeric). Changed prior to commit: http://reviews.llvm.org/D21290?vs=60525=60575#toc Repository: rL LLVM

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

2016-06-13 Thread Benjamin Kramer via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. Thanks! http://reviews.llvm.org/D21290 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

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

2016-06-13 Thread Eric Liu via cfe-commits
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