[PATCH] D70225: [clangd] Simplify the code in XRefs

2019-11-14 Thread Ilya Biryukov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. ilya-biryukov marked an inline comment as done. Closed by commit rG5a9547b00709: [clangd] Simplify the code in Index::refs (authored by ilya-biryukov). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[PATCH] D70225: [clangd] Simplify the code in XRefs

2019-11-14 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov marked 2 inline comments as done. ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/index/Merge.cpp:116 // files. - More |= Static->refs(Req, [&](const Ref &O) { if (DynamicIndexFileURIs.count(O.Location.FileURI)) ho

[PATCH] D70225: [clangd] Simplify the code in XRefs

2019-11-14 Thread Haojian Wu via Phabricator via cfe-commits
hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Thanks, the title is a bit confusing, the modified code is about the index, not XRefs. Comment at: clang-tools-extra/clangd/index/Merge.cpp:116 // files. - More |= Stati

[PATCH] D70225: [clangd] Simplify the code in XRefs

2019-11-14 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. Build result: pass - 60074 tests passed, 0 failed and 729 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70225: [clangd] Simplify the code in XRefs

2019-11-14 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. ilya-biryukov added a reviewer: hokein. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, MaskRay. Herald added a project: clang. While here, also fix potential UB in MergeIndex. Thanks Kadir for finding this! Repository: rG LLVM Github Mon