[PATCH] D88204: [clangd] Drop path suffix mapping for std symbols

2020-10-12 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 297528. kadircet added a comment. - Only drop STL mapping instead of getting rid of suffix mapping completely. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88204/new/ https://reviews.llvm.org/D88204 Files:

[PATCH] D88204: [clangd] Drop path suffix mapping for std symbols

2020-10-12 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet marked an inline comment as done. kadircet added a comment. > We don't have an alternative database for these. Maybe we should keep the > mechanism but stop using it for the standard library? What do you think? Yes i think you are right. Just dropping STL mappings from the list. Repos

[PATCH] D88204: [clangd] Drop path suffix mapping for std symbols

2020-10-09 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Taking another look at the header list, there are a couple of classes of symbols beyond c/c++ standard library. We don't have an alternative database for these. Maybe we should keep the mechanism but stop using it for the standard library? What do you think? ==

[PATCH] D88204: [clangd] Drop path suffix mapping for std symbols

2020-09-24 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, jfb, arphaman. Herald added a project: clang. kadircet requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. As mentioned in D88144