[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

ChuanqiXu added a subscriber: v.g.vassilev. ChuanqiXu added a comment. @sammccall @nridge in a chat, @v.g.vassilev mentioned that it may be possible to make clangd for modules a GSoC project. I feel it is pretty good idea. How do you think about it? Repository: rG LLVM Github Monorepo

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

nridge added a comment. In D137770#3945095 , @spartacoos wrote: > I realised this is an active issue that's not being worked on. So, I want to > put myself forward to help implement this new module system for clangd once > you have settled on a

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

ChuanqiXu added a comment. In D137770#3945095 , @spartacoos wrote: > Hi @nridge @ChuanqiXu, I've been dealing with some issues trying to get > modules working with clangd only to end up hacking some flimsy solutions > which inevitably breakdown under

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

spartacoos added a comment. Hi @nridge @ChuanqiXu, I've been dealing with some issues trying to get modules working with clangd only to end up hacking some flimsy solutions which inevitably breakdown under some edge cases, those problems led me to this thread

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

nridge added a comment. In D137770#3919327 , @ChuanqiXu wrote: > And we need the compiler (clang) to parse the module file from different > versions. I think it's more likely that we'll need to get clangd to build the modules itself (or by using a

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

ChuanqiXu planned changes to this revision. ChuanqiXu added a comment. @sammccall Thanks for you professional and valuable input. It is very helpful. From the perspective of clangd, it looks not good indeed that the user can't get a stable support for modules. > This situation is not at all

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

sammccall added a comment. The status quo for modules on clangd is all flavors are completely unsupported: https://github.com/clangd/clangd/issues/1293. I don't think we should encourage people to set up workflows that are known not to work properly and will probably break in future. It's true

[PATCH] D137770: [docs] Introduce clangd part in StandardCPlusPlusModules

ChuanqiXu created this revision. ChuanqiXu added reviewers: sammccall, nridge. Herald added subscribers: kadircet, arphaman. Herald added a project: All. ChuanqiXu requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang.