[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-16 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Thanks Hans! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87710/new/ https://reviews.llvm.org/D87710 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm

[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-16 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. Pushed to 11.x as 4a26e3b33798424dc5a4843f7b29a617bef81656 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87710/new/ https://reviews.llvm.org/D87710 _

[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-16 Thread Adam Czachorowski via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7029e5d4ca20: [clangd] Actually parse Index section of the YAML file. (authored by adamcz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-16 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. In D87710#2275933 , @sammccall wrote: > Argh, sorry. I think this might be too late to get into 11 - we're in the > "soon=final" stage and maybe any changes at all cause logistical problems? :-( > > @hans to confirm - this is a featu

[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-16 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a subscriber: hans. sammccall added a comment. This revision is now accepted and ready to land. Argh, sorry. I think this might be too late to get into 11 - we're in the "soon=final" stage and maybe any changes at all cause logistical problems? :-

[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-15 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added a comment. oh shoot, thanks for catching this. i'll leave the stamp to sam, just came here to remind that this needs to be cherry-picked into release branch :/ Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87710/new/ https://reviews

[PATCH] D87710: [clangd] Actually parse Index section of the YAML file.

2020-09-15 Thread Adam Czachorowski via Phabricator via cfe-commits
adamcz created this revision. adamcz added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous. Herald added a project: clang. adamcz requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. This fixes a bug in dbf486c0de